feat(fastbreakv1): use references across the board for FastBreakContractV1#294
Merged
Merged
Conversation
…age for games and runs. * Introduce helper functions for managing year-specific storage, and update existing methods to support both legacy and new data structures. * Ensure seamless access to game and run data across different years while maintaining backward compatibility.
added 6 commits
November 6, 2025 13:36
…es. Adjusted comments to reflect updated contract size and modified digest for asset integrity.
…ntroduced new methods for retrieving and setting games and runs by year, ensuring backward compatibility with legacy data structures. Updated existing scripts to utilize year-based queries for game and run data, improving data management and retrieval efficiency.
…retrieval. Updated methods to utilize a unified approach for accessing game and run data, prioritizing current year, previous year, and legacy storage. Removed deprecated code and improved comments for clarity.
…ed digest for asset integrity. Adjusted comments to reflect the latest changes.
…hods and streamline data retrieval. Updated functions to utilize a unified approach for accessing game and run data, ensuring backward compatibility. Removed deprecated scripts and improved comments for clarity.
… asset data sizes. Adjusted comments for clarity and modified digest values for asset integrity. Streamlined data retrieval methods to enhance performance and maintain backward compatibility.
joshuahannan
approved these changes
Nov 7, 2025
…emoving unnecessary type casting. Updated functions to return references directly, enhancing performance and maintaining backward compatibility.
chumeston
changed the title
added 2 commits
November 7, 2025 13:15
…ed digest for asset integrity. Adjusted comments to reflect the latest changes in asset data size and structure.
…tracts, ensuring compatibility with the latest deployments. Adjusted type casting for submission retrieval to enhance safety and clarity.
loic1
approved these changes
Nov 10, 2025
Contributor
loic1
left a comment
loic1
left a comment
There was a problem hiding this comment.
nice! just added a few non-blocking questions
added 2 commits
November 10, 2025 09:57
…rted contracts, ensuring compatibility with upcoming deployments. This change prepares the contract for integration with the latest versions of dependencies.
…thods with a more efficient key-based approach. Updated comments for clarity and adjusted asset data size and digest for integrity. This change enhances performance and maintains compatibility with existing deployments.
6 tasks
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
4 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.
#GOLD-517