fix: resolve issues #66 #80 #86 #92 assigned to solomon35-stack - #166
Merged
dzekojohn4 merged 2 commits intoJul 27, 2026
Merged
Conversation
…nityChainxx#92 assigned to solomon35-stack fix(UnityChainxx#92): guard mock data in puzzleReviewService behind USE_MOCKS flag - Removed sensitive fields (ipAddress, deviceInfo) from mock data - Mocks only load when NEXT_PUBLIC_USE_MOCKS=true - Production bundle ships with empty defaults fix(UnityChainxx#86): wrap createMatch writes in TypeORM transaction - Inject DataSource and use manager.transaction() for atomicity - Prevents partial state when the process crashes between saving the match and updating queue entries fix(UnityChainxx#80): ShareButton shares canonical URL instead of current page - Replaced window.location.href with configurable NEXT_PUBLIC_SHARE_URL - Falls back to NEXT_PUBLIC_APP_URL or https://stellarhunts.com - Prevents leaking puzzle/game IDs via shared URLs feat(UnityChainxx#66): add Vitest + Testing Library test suite to frontend - Added test and test:watch scripts to frontend/package.json - Created vitest.config.mjs with jsdom environment - Added 17 unit tests for useReferral hook - Updated CONTRIBUTING.md with frontend test instructions
|
@solomon35-stack Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix(#92): guard mock data in puzzleReviewService behind USE_MOCKS flag
fix(#86): wrap createMatch writes in TypeORM transaction
fix(#80): ShareButton shares canonical URL instead of current page
feat(#66): add Vitest + Testing Library test suite to frontend
Summary
Changes
Testing
Related Issues
closes #66
closes #80
closes #86
closes #92