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.

Sign up for GitHub

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

Jump to bottom

move(visitor): expose underlying byte slice #19517 #4783

Open
bingyanglin opened this issue Jan 15, 2025 · 1 comment
Open

move(visitor): expose underlying byte slice #19517 #4783

bingyanglin opened this issue Jan 15, 2025 · 1 comment
Assignees
@bingyanglin
Labels
node Issues related to the Core Node team
Milestone
Mainnet

Comments

@bingyanglin
Copy link
Contributor

bingyanglin commented Jan 15, 2025 •
edited
Loading

move(visitor): expose underlying byte slice #19517

This is the PR contains the changes of #4773 #4774 #4775 and #4681, which modified and enhanced the visitor logic.
Note that in the SUI upstream, the PRs of the above four issues (and many more) are merged to amnn/mv-visitor-refactors first, then the amnn/mv-visitor-refactors is merged to the SUI main branch by #19517.
@bingyanglin bingyanglin self-assigned this Jan 15, 2025
@bingyanglin bingyanglin added the node Issues related to the Core Node team label Jan 15, 2025
@bingyanglin bingyanglin added this to the Mainnet milestone Jan 15, 2025
@bingyanglin
Copy link
Contributor Author

After more investigation, the crates/iota-graphql-rpc/src/types/dynamic_fields.rs has four prior dependancies

Which should be merged first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bingyanglin bingyanglin

Labels
node Issues related to the Core Node team
Projects
None yet
Milestone
Mainnet
Development

No branches or pull requests
1 participant
@bingyanglin