-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(streaks): Add streak page from real code and bd. #20
Conversation
🚀 Preview for this PR is available at: https://stepwise-bdab7--pr-20-biftjgzr.web.app 🧪 Test Results🗂 Test Summary: Test Files 2 passed (2) 🎉 All tests passed successfully! |
Hold on approving this for now I will review it. |
- Moved streak logic from context to streamlined streakUtils.js for better modularity - Removed redundant streakUtils and taskCompletion utils in firebase directory - Updated Streak.jsx to improve rendering logic and performance, optimized tileContent for calendar - Refactored useGoalsUpdater.js to consolidate goal and streak updates - Directly use context for streak in Header
🚀 Preview for this PR is available at: https://stepwise-bdab7--pr-20-biftjgzr.web.app 🧪 Test Results🗂 Test Summary: Test Files 2 passed (2) 🎉 All tests passed successfully! |
It's all good now, please go ahead and approve this commit. Remember to do |
🚀 Preview for this PR is available at: https://stepwise-bdab7--pr-20-biftjgzr.web.app 🧪 Test Results🗂 Test Summary: Test Files 2 passed (2) 🎉 All tests passed successfully! |
File Modifications
common/Header
: Use real data from DB instead of hard code.hooks/useGoalsUpdater
: Refactored to streamline goal and streak updates, update streak every time task status update.pages/Streak
: Use MUI calendar implement real DB data.utils/streakUtils
: Consolidated streak-related functions and logic into this utilityDependency Updates
npm install
to ensure all dependencies are up-to-date.Screen.Recording.2024-11-05.at.9.04.40.AM.mov