Repository navigation
Update documentation and review sprint bugs - #16
Merged
CoderMariusz merged 2 commits intoDec 3, 2025
Merged
Conversation
- Add smooth camera position animation (Issue #4) - Create game_config.dart with centralized constants (Issue #5) - Make Firebase optional with try-catch fallback (Issue #6) - Update sprint-status.yaml to show Epic 0-1 as done (Issue #7) - Extract magic numbers to constants in BuildingComponent (Issue #8) - Update CLAUDE.md, FILE-MAP.md, EPIC-1-ISSUES.md All critical bugs were already fixed in code (documentation was outdated). Epic 1 is now production ready.
- Extract hardcoded constants in PlayerEconomy to static class constants - Add minimum collection cooldown (60s) to prevent spam-tapping - Add tooSoon/secondsUntilNextCollection to CollectResourcesResult - Use CameraConstants as defaults in GridCameraConfig - Create ResourceIds class for type-safe resource identifiers - Update main.dart to use ResourceIds instead of magic strings - Add CollectionConstants for collection mechanics config - Update BuildingComponent to show cooldown message when tapped too soon - Add color parameter to FloatingTextComponent for different message types Maintains Clean Architecture: domain layer uses inline constants, game/config layers can import from game_config.dart.
CoderMariusz
deleted the
claude/update-docs-review-bugs-01C6VYdyKqt7GFaSFZEbN264
branch
December 4, 2025 09:32
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.
No description provided.