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

Update logic for coloring chapters' last action #211

Merged
merged 2 commits into from
Jun 3, 2024
Merged

Conversation

alexsapps
Copy link
Collaborator

No description provided.

… from Michelle

Tested by manually setting last event dates to be 29,30,58,59 days ago in local database and checking the color.
@alexsapps alexsapps requested a review from jakehobbs May 30, 2024 06:07
@@ -871,10 +871,10 @@ export default Vue.extend({
if (time.isValid()) {
c = 'is-danger';
}
if (time.isAfter(dayjs().add(-2, 'month'))) {
if (time.isAfter(dayjs().add(-58, 'day'))) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yuck forgot we were still using dayjs in this 🤣

fine thought for now obvs

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol. noted dayjs is yuck.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not as bad as moment. i didn't realize it's still being updated, but it is. i prefer date-fns these days though, and it has a smaller bundle size.

@alexsapps alexsapps merged commit 81fab1a into master Jun 3, 2024
1 check passed
@alexsapps alexsapps deleted the alex/chaptercolor branch June 3, 2024 03:07
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 this pull request may close these issues.

2 participants