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

Refactor and optimize collection update mechanism #21

Merged
merged 3 commits into from
Dec 8, 2023

Conversation

reedoooo
Copy link
Owner

@reedoooo reedoooo commented Dec 8, 2023

No description provided.

- Enhanced PATCH endpoint in router to ensure comprehensive data integrity checks and preservation of key card attributes like quantity, priceHistory, and latestPrice.
- Implemented useEffect in context to automatically update local and server state when cards are added, removed, or their quantity changes in a collection.
- Improved cron job logic to trigger only when specific conditions are met, enhancing efficiency.
- Fixed issue with disappearing header title in the PortfolioContent component during chart re-renders.
…handling

- Implement StatisticsProvider to use currentChartDataSets2 from CollectionStore.
- Create and utilize waitForChartDataUpdate function for async context updates.
- Refactor ChartProvider to handle dynamic timeRange updates.
- Optimize StatisticsContext with useMemo for efficient data processing.
- Add error handling and performance considerations in async operations.
- Adjust components to consume updated context values and handle state changes.
- Implemented advanced styling for SelectDeckList component, including modern button styling, improved typography, and icon enhancements.
- Added responsive pagination to the ProductGrid component using Material-UIs Pagination component.
- Updated the CartProvider to fix an infinite loop issue and optimized context value updates.
- Refactored and enhanced the updateExistingCardInUserCollection function to ensure synchronization with CardInCollection model and users collection data.
- Cleaned and optimized various React components for better performance and readability.
@reedoooo reedoooo merged commit 5f72ed3 into main Dec 8, 2023
1 check failed
Copy link

netlify bot commented Dec 8, 2023

Deploy Preview for tcg-store ready!

Name Link
🔨 Latest commit a053a56
🔍 Latest deploy log https://app.netlify.com/sites/tcg-store/deploys/65729102e821a800084b7062
😎 Deploy Preview https://deploy-preview-21--tcg-store.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 8, 2023

Deploy Preview for enhanced-cardstore ready!

Name Link
🔨 Latest commit a053a56
🔍 Latest deploy log https://app.netlify.com/sites/enhanced-cardstore/deploys/65729102bca4a9000831d400
😎 Deploy Preview https://deploy-preview-21--enhanced-cardstore.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

1 participant