-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update GraphQL queries to reflect new schema changes #795
Conversation
Adjusted `AccountInfo`, `login`, and `create` queries to align with the new schema, including changes to account controllers and finalizing login mutations. These updates ensure proper querying and mutation according to the updated backend structure.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Adjusted `AccountInfo`, `login`, and `create` queries to align with the new schema, including changes to account controllers and finalizing login mutations. These updates ensure proper querying and mutation according to the updated backend structure.
729060f
to
fa7b559
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #795 +/- ##
=======================================
Coverage 68.51% 68.51%
=======================================
Files 4 4
Lines 108 108
Branches 36 36
=======================================
Hits 74 74
Misses 32 32
Partials 2 2 ☔ View full report in Codecov by Sentry. |
@@ -69,7 +69,7 @@ function Form({ | |||
appId: origin, | |||
chainId, | |||
rpcUrl, | |||
address, | |||
address: controllers[0].address, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@broody we should check how we can support multiple controllers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah for sure, just getting gql to work for now, multi controller support and also multi signers will be a bigger PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fair
b1e7ce8
to
68fcba5
Compare
8e8fffa
to
6dfcd63
Compare
Adjusted
AccountInfo
,login
, andcreate
queries to align with the new schema, including changes to account controllers and finalizing login mutations. These updates ensure proper querying and mutation according to the updated backend structure.