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

#230 Add Streaks component #353

Merged
merged 2 commits into from
Apr 3, 2022
Merged

#230 Add Streaks component #353

merged 2 commits into from
Apr 3, 2022

Conversation

zwan702
Copy link
Collaborator

@zwan702 zwan702 commented Apr 2, 2022

1648893434(1)
This helps the user in more easily determining when they will be the busiest with meetups and encounters, and can help them to schedule better.

This feature will be present in the homepage of the app, and will track all the encounters input by the user and use that to build the streaks.

Acceptance Criteria

  • Develop a streaks component for use in the frontend
  • Modify the homepage to include the new feature
  • Implement backend api and calls to allow streaks to properly display the needed data

Implementation details

New api calls in the backend that utilises the current encounters and transforms that to data that can populate the streaks feature.

New dependencies

react-calendar-heatmap
react-tooltip

need npm i react-calendar-heatmap
and
npm i react-tooltip
Copy link
Collaborator

@Dj5822 Dj5822 left a comment

Choose a reason for hiding this comment

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

Looks good to me, although we seem to be missing tests.

@zwan702 zwan702 merged commit b1328b5 into se701team3:main-a2 Apr 3, 2022
abis555 pushed a commit to abis555/Forgettable that referenced this pull request Apr 5, 2022
* Add Streaks component

need npm i react-calendar-heatmap
and
npm i react-tooltip

* remove dead code, add some comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants