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

[recnet-web] Display cycle in feeds page & consolidate date picker component #320

Merged
merged 8 commits into from
Oct 9, 2024

Conversation

swh00tw
Copy link
Collaborator

@swh00tw swh00tw commented Oct 4, 2024

Description

This PR mainly address two issues

  1. Duplicate code in CutoffDatePicker and in DatePicker. This PR refactor the code duplication in CutoffDatePicker and extend DatePicker functionality.
  2. Currently, the /feeds page doesn't show the current cycle. When navigating to old cycles, it's hard to know which cycle we are viewing currently.

Related Issue

Notes

Test

The CutoffDatePicker should behave the same as in the past.

  • Can only select Tuesday.

Screenshot 2024-10-04 at 3 31 41 PM

We could see a "current cycle" indicator in feeds page like this
Screenshot 2024-10-04 at 3 32 45 PM

TODO

  • Paste the testing link
  • Clear console.log or console.error for debug usage
  • Update the documentation recnet-docs if needed
  • Version bump in package.json if needed

Copy link

vercel bot commented Oct 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
recnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 9, 2024 4:11am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
recnet-docs ⬜️ Ignored (Inspect) Visit Preview Oct 9, 2024 4:11am

@swh00tw swh00tw merged commit 7aff997 into dev Oct 9, 2024
4 checks passed
swh00tw added a commit that referenced this pull request Oct 9, 2024
## RecNet auto-release action
This is an auto-generated PR by recnet-release-action 🤖
Please make sure to test your changes in staging before merging. 
## Related Issues
- #60
- #324
- #317
- #253
- #254
## Related PRs
- #322
- #325
- #320
- #311
## Staging links
recnet-web:
[https://vercel.live/link/recnet-git-dev-recnet-542617e7.vercel.app](https://vercel.live/link/recnet-git-dev-recnet-542617e7.vercel.app)
recnet-api:
[https://dev-api.recnet.io/api](https://dev-api.recnet.io/api)
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