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

chore: Pull Upstream Changes but keep schemars #26

Closed
wants to merge 119 commits into from
Closed

chore: Pull Upstream Changes but keep schemars #26

wants to merge 119 commits into from

Conversation

DaughterOfMars
Copy link
Contributor

Description

Pulls upstream changes as in #23 but excludes the removal of schemars implementations.

bmwill and others added 30 commits August 21, 2024 09:12
@bmwill
type_tag: include source string in parse error
59ba52a
@stefan-mysten
sui-graphql-client: introduce initial implementation a sui GraphQL cl…
2b80475 
…ient (#4)
@stefan-mysten
graphql: update to latest graphql schema (#7)
516028a
@stefan-mysten
graphql: add a few tests (#8)
308347f
@bmwill
coin: use Cow for coin_type
b6d1873
@bmwill
signature: add bytes and base64 to/from methods for UserSignature
58e5068
@bmwill
types: introduce Intent types
a94f1d4
@bmwill
types: introduce PersonalMessage
9e851f4
@bmwill
hash: add methods for getting the signing digest
92c06bc 
Add methods for Transaction and PersonalMessage for calculating the
signing digest which is used as the message to sign against in Sui.
@bmwill
crypto: remove unused private key types
489ed2f
@bmwill
sui-sdk-crypto: introduce crate to handle signing and verifying
8b868c4
@bmwill
crypto: introduce secp256r1 support
2c8f5a5
@bmwill
crypto: tweak dependencies and default features
5050f93
@bmwill
crypto: introduce secp256k1 support
3fcbb7e
@bmwill
crypto: use wasm compatible k256 library for secp256k1 support
646aee1
@bmwill
crypto: introduce zklogin verifier support
cc2c6bc
@bmwill
chore: rename sui-sdk to sui-sdk-types
263e90a
@bmwill
chore: rename sui-sdk-crypto to sui-crypto
198a68c
@bmwill
sui-graphql-client: ignore test_object_query test
7d2f81b 
Ignore the sui-graphql-client::test_object_query test since its has been
unreliable due to the dependency on an extrnal running network.
@bmwill
fix doc link in readme
7b64009
@bmwill
prepare sui-sdk-types-0.0.1 and sui-crypto-0.0.1
18bae13
@stefan-mysten
chore: update crates to latest versions (#19)
7363454
@stefan-mysten
sui-graphql-rpc: add balance function (#13)
dff3361
@stefan-mysten
sui-graphql-client: add coins function (#12)
8339695
@stefan-mysten
sui-graphql-client: introduce a faucet client (#11)
2374813
@stefan-mysten
sui-graphql-client: use 0x0 as sender for event if none (#20)
c055420
@stefan-mysten
sui-graphql-client: add method for querying total supply of a coin (#17)
74b91c1
@stefan-mysten
sui-graphql-client: add execute transaction query (#22)
bc65d4a
@stefan-mysten
sui-graphql-client: ignore events test due to graphql server failures (…
e3b24ad 
…#26)
@stefan-mysten
sui-graphql-client: add active validators query function (#21)
0e0ff8e
muXxer and others added 18 commits February 17, 2025 13:18
@muXxer @DaughterOfMars @thibault-martinez
chore: rustfmt (#3)
ebefd0f 
* chore: rustfmt

* chore: add dprint config

* Format with nightly

* fix CI

* checkout

* install nightly

* reformat

* remove ignore external-crates

---------

Co-authored-by: Chloe Martin <[email protected]>
Co-authored-by: Thibault Martinez <[email protected]>
@DaughterOfMars
Add EndOfEpochData::epoch_supply_change field
a56c0bd 
update fixture

fmt
@alexsporn @DaughterOfMars
chore(types): cleaned up removed and renamed transaction types (#5)
f658261
@valeriyr @DaughterOfMars
feat: remove the DenyListStateCreate transaction (#7)
7abf263
@piotrm50 @DaughterOfMars
chore(cleanup): Remove legacy MultiSig (#8)
cdb2d8a 
* Add .idea/ to .gitignore

* Remove legacy MultiSig

* Remove test of legacy fixture
@Thoralf-M @DaughterOfMars
feat!: rename types for iota PR iotaledger/iota#3499 (#9)
2106f05
@Thoralf-M @DaughterOfMars
feat!: remove TransactionEffectsV1 and rename V2 to V1
70178eb
@Thoralf-M @DaughterOfMars
Update tests
ec65a0b
@DaughterOfMars
fix: Add events to genesis transaction
0aad1f8 
more renames

increase limit
@Thoralf-M @DaughterOfMars
fix: update fixtures
f562884
@DaughterOfMars
fix(CI): Fix workflow triggers (#21)
ab64609
@jkrvivian @DaughterOfMars
refactor: Remove has_public_transfer in MoveStruct
301bbe2
@jkrvivian @DaughterOfMars
fix: Update serialization test bytes
d47ec7c
@Thoralf-M @DaughterOfMars
add crate to update transaction fixtures and also update them
3e0ea9d
@DaughterOfMars
feat(protocol): Add computation_cost_burned field to GasCostSummary (#20
fdf6d63 
)
@muXxer @DaughterOfMars
rename: fix capitalization (Iota to IOTA) (#24)
7a76797
@DaughterOfMars
chore: Fix some typos, CI, docs, urls and re-add signing fns that wer…
1af3664 
…e missed
@DaughterOfMars
Merge branch 'develop' into dev-tools/pull-upstream-3
bd62950
@DaughterOfMars DaughterOfMars force-pushed the dev-tools/pull-upstream-3 branch from 03a3675 to bd62950 Compare February 17, 2025 19:26
@DaughterOfMars DaughterOfMars mentioned this pull request Feb 21, 2025
Closed
@DaughterOfMars DaughterOfMars marked this pull request as ready for review February 21, 2025 13:40
@DaughterOfMars DaughterOfMars mentioned this pull request Feb 21, 2025
Open
@Thoralf-M
Copy link
Member

CI fails

@DaughterOfMars
Copy link
Contributor Author

CI fails

I don't think it's ever succeeded?

@Thoralf-M
Copy link
Member

Thoralf-M commented Feb 21, 2025 •
edited
Loading

I don't think it's ever succeeded?

It did at some point, but now it doesn't even compile, at least this should work

error: this file contains an unclosed delimiter
   --> crates/iota-sdk-types/src/hash.rs:379:3
    |
294 | impl crate::ObjectId {
    |                      - unclosed delimiter
...
301 |     pub fn derive_id(digest: crate::TransactionDigest, count: u64) -> Self {
    |                                                                            - this delimiter might not be properly closed...
...
336 | }
    | - ...as it matches this but it has different indentation
...
379 | }
    |  ^
    ```

DaughterOfMars pushed a commit that referenced this pull request Feb 24, 2025
@stefan-mysten @DaughterOfMars
sui-graphql-client: ignore events test due to graphql server failures (…
95ebb1a 
…#26)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thibault-martinez thibault-martinez Awaiting requested review from thibault-martinez

@Thoralf-M Thoralf-M Awaiting requested review from Thoralf-M

@Alex6323 Alex6323 Awaiting requested review from Alex6323

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.