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

Already on GitHub? Sign in to your account

AO3-4138 make reading history more linguistically accessible #3081

Merged
merged 3 commits into from Nov 25, 2017

Conversation

Projects
None yet
5 participants
Contributor

pupupuri commented Sep 26, 2017

Issue

https://otwarchive.atlassian.net/browse/AO3-4138

Purpose

in reading history, changes "last viewed: (date)" and "viewed: (number)" to "last visited" and "visited"

Testing

please go to your reading history and confirm that the text bar under each work in your history now says "last visited: (date)" and "visited: (number)"

References

the initial pr was closed; sorry about that

sarken approved these changes Sep 26, 2017

Looks good!

If you have time, would you change the syntax from :date => set_format_for_date(reading.last_viewed) to date: set_format_for_date(reading.last_viewed) in the blurb file?

If you don't have time, no worries!

Contributor

pupupuri commented Sep 26, 2017

no worries! i can do that.

pupupuri added some commits Sep 27, 2017

Owner

sarken commented Sep 28, 2017

Terrific -- thank you!

@otw-deploy otw-deploy closed this Oct 31, 2017

@zz9pzza zz9pzza reopened this Oct 31, 2017

@elzj elzj merged commit b284f6e into otwcode:master Nov 25, 2017

3 of 4 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
Scrutinizer No new issues
Details
codeclimate All good!
Details
hound No violations found. Woof!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment