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

Add Year filter #677

Merged
merged 10 commits into from
Sep 15, 2021
Merged

Add Year filter #677

merged 10 commits into from
Sep 15, 2021

Conversation

litelime
Copy link
Contributor

No description provided.

@whyboris
Copy link
Owner

Thank you for your contribution 🙇
I'll try to find time this week to test it out and merge it in 🤝

Copy link
Owner

@whyboris whyboris left a comment

Choose a reason for hiding this comment

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

Thank you for the fine work! 🎉
Thanks for your patience -- took me a while to get to review it 😓
I might make some minor (mostly whitespace) adjustments to some of your code after I merge it 🤝

@whyboris whyboris merged commit 051014b into whyboris:main Sep 15, 2021
@whyboris
Copy link
Owner

Did some minor whitespace fixes, alphabetization, and added some missing ; 🤝 #701 🤝

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.

None yet

2 participants