Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rethink "Seen on [date]" in list of seen movies #105

Closed
wtimme opened this issue Jul 14, 2019 · 1 comment · Fixed by #108
Closed

Rethink "Seen on [date]" in list of seen movies #105

wtimme opened this issue Jul 14, 2019 · 1 comment · Fixed by #108

Comments

@wtimme
Copy link
Contributor

wtimme commented Jul 14, 2019

Is your feature request related to a problem? Please describe.
When I start using the app, I want to put movies that I have already seen on the (empty) "History" list. After adding a movie, the list says "Seen on [current date]". This is not correct - I have only marked it as seen at that date.

seen

(I've seen this movie a couple of years back - not just some minutes ago.)

Describe the solution you'd like
For the MVP, I propose we rephrase this to "Added on [current date]".

Describe alternatives you've considered
I might not be getting the purpose of this "History" screen.

Additional context
Personally, I initially thought of this as a list of movies that I watched and can recommend. Especially since the "call to action" in the empty state is worded like this:

You don't have any movies you have already seen on this list.
Go ahead and add a new movie.

As a new user, this sounds like I could have a collection of movie recommendations.

@fbernutz
Copy link
Member

fbernutz commented Jul 15, 2019

I really like the idea of just changing the wording. There is another issue open #90 where you could change the date on which you watched the movie. But until this feature is implemented, I love the wording suggestion! Do you want to make a PR for the wording change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants