Skip to content

Sync dynamic data getter#158

Closed
EndymionJkb wants to merge 3 commits intodev/modifier-shufflefrom
dynamic-data
Closed

Sync dynamic data getter#158
EndymionJkb wants to merge 3 commits intodev/modifier-shufflefrom
dynamic-data

Conversation

@EndymionJkb
Copy link
Collaborator

Description

The base PR #157 changes some logic to improve accuracy: this updates the dynamic data getter to reflect the changes.

Note: This assumes we are able to change the dynamic data struct. If we cannot do that (because there's too much API usage tied to it), then I'll update this to use the same ABI (but still be more accurate).

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Dependency changes
  • Code refactor / cleanup
  • Optimization: [ ] gas / [ ] bytecode
  • Documentation or wording changes
  • Other

Checklist:

  • The diff is legible and has no extraneous changes
  • Complex code has been commented, including external interfaces
  • Tests have 100% code coverage
  • The base branch is either main, or there's a description of how to merge

Issue Resolution

@EndymionJkb
Copy link
Collaborator Author

Overtaken by events; will still need to do some form of this, but maybe in the main PR (won't require extra fields).

@EndymionJkb EndymionJkb closed this Mar 4, 2026
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