Skip to content

Commit

Permalink
graphql: ObjectOwner::Parent exposed as Owner (MystenLabs#19785)
Browse files Browse the repository at this point in the history
## Description

Now that we no longer expose wrapped objects, we need a way to expose
the addresses of object parents when they are other objects that have
been wrapped.

## Test plan

Updated tests:

```
sui$ cargo nextest run -p sui-graphql-e2e-tests
sui$ cargo nextest run -p sui-graphql-rpc
```

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ ] Protocol: 
- [ ] Nodes (Validators and Full nodes): 
- [ ] Indexer: 
- [ ] JSON-RPC: 
- [x] GraphQL: Change `Parent.parent` from an `Object` to an `Owner`.
Although it's guaranteed to be an object if it exists, it may be
wrapped, in which case it will not exist. Exposing it as an Owner allows
queries to extract its ID and also fetch other dynamic fields from it
even if it is wrapped.
- [ ] CLI: 
- [ ] Rust SDK:
- [ ] REST API:
  • Loading branch information
amnn authored Oct 10, 2024
1 parent e97402c commit 4d63e50
Show file tree
Hide file tree
Showing 19 changed files with 550 additions and 157 deletions.
5 changes: 5 additions & 0 deletions .changeset/hot-stingrays-press.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@mysten/graphql-transport': patch
---

Update GraphQL transport layer to accommodate change in schema
Loading

0 comments on commit 4d63e50

Please sign in to comment.