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.

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

[wallet-ext] Add Origin Byte NFTs (v0.5.0) support #5564

Closed
wants to merge 16 commits into from

Conversation

oribyts
Copy link
Contributor

@oribyts oribyts commented Oct 26, 2022

Add support of new version of OriginByte NFT protocol.

SUI-wallet.mov

The main difference with previous PR is adding external library and response parser, which will be helpful if the data structure will change again.

lxfind and others added 16 commits October 18, 2022 13:55
* use genesis.blob to set up remote bench

* add AuthorityAggregatorBuilder
Narwhal consensus latency is 5~6s in private testnet. Using a timeout of 5s is too low. Increasing the default value to avoid timeout errors. These defaults are used by AuthorityClients.
* revert flat vm fee while we make it easier for users
* splitting local network from client cli

* Removing gateway mentions

* Updates for gateway deprecation

* Update json-rpc.md

* Updates for gateway deprecation

* Updates for gateway deprecation

* Update sui-json.md

* Updates for gateway deprecation

* Update sui-local-network.md

* Update sui-local-network.md

* Update doc/src/build/comms.md

Co-authored-by: Lu Zhang <[email protected]>

* Update doc/src/build/devnet.md

Co-authored-by: Lu Zhang <[email protected]>

* Update doc/src/build/fullnode.md

Co-authored-by: Lu Zhang <[email protected]>

* Update doc/src/build/fullnode.md

Co-authored-by: Lu Zhang <[email protected]>

* Update doc/src/build/fullnode.md

Co-authored-by: Lu Zhang <[email protected]>

* Updates per tech review

* Update fullnode.md

* Update fullnode.md

* Update json-rpc.md

* Update fullnode.md

* Update fullnode.md

* Additional changes for gateway deprecation

Co-authored-by: Lu Zhang <[email protected]>
@oribyts oribyts changed the title [wallet] Add Origin Byte NFTs (v0.5.0) support [wallet-ext] Add Origin Byte NFTs (v0.5.0) support Oct 26, 2022
@randall-Mysten
Copy link
Contributor

Hi @oribyts, please open PRs to contribute against the main branch rather than devnet. Looks like you may need to rebase to release .13 as well

@oribyts
Copy link
Contributor Author

oribyts commented Nov 4, 2022

@randall-Mysten I've decided to create another PR to prevent conflicts resolving (PR is not a big and was easier to move code): #5845

@oribyts oribyts closed this Nov 4, 2022
@randall-Mysten
Copy link
Contributor

Hi @oribyts Please also submit PRs to main instead of devnet. Thank you. Create a fork of Mysten\sui main and then submit create the PR from your fork.

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.

9 participants