Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

enhancement: transparently add internal types for GraphQL connection specification #1403

Merged
merged 9 commits into from
Oct 17, 2023

Conversation

deekerno
Copy link
Contributor

@deekerno deekerno commented Oct 9, 2023

Description

This PR transparently adds internal types to be used as part of the GraphQL connections specification. A user will not have to add these types to their schema as they will be generated and added to their schema for each field that uses a list type. The next PR will add functionality for creating introspection information, which will expose these types to clients.

Testing steps

Added new unit tests. CI should pass.

Manual Testing

Run the fuel-explorer example and ensure that it works properly.

@deekerno deekerno self-assigned this Oct 9, 2023
@deekerno deekerno force-pushed the deekerno/add-transparent-pagination-types-to-schema branch from 82ddb1b to 283a200 Compare October 9, 2023 03:39
@deekerno deekerno changed the title enhancement: implement types for connection specification enhancement: transparently add types for GraphQL connection specification Oct 9, 2023
@deekerno deekerno changed the title enhancement: transparently add types for GraphQL connection specification enhancement: transparently add schema types for GraphQL connection specification Oct 9, 2023
@deekerno deekerno force-pushed the deekerno/add-transparent-pagination-types-to-schema branch 7 times, most recently from e46e4ee to f0c6d56 Compare October 16, 2023 16:10
@deekerno deekerno force-pushed the deekerno/add-transparent-pagination-types-to-schema branch from f0c6d56 to c2fbd80 Compare October 16, 2023 16:28
@deekerno deekerno changed the title enhancement: transparently add schema types for GraphQL connection specification enhancement: transparently add internal types for GraphQL connection specification Oct 16, 2023
@deekerno deekerno marked this pull request as ready for review October 16, 2023 20:04
lostman
lostman previously approved these changes Oct 17, 2023
Copy link
Contributor

@ra0x3 ra0x3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deekerno

  • Left some feedback
  • I forget, were there supposed to be insta snapshot tests in this PR as well?

packages/fuel-indexer-lib/src/graphql/mod.rs Show resolved Hide resolved
packages/fuel-indexer-lib/src/graphql/mod.rs Outdated Show resolved Hide resolved
packages/fuel-indexer-lib/src/graphql/mod.rs Outdated Show resolved Hide resolved
packages/fuel-indexer-lib/src/graphql/mod.rs Show resolved Hide resolved
packages/fuel-indexer-lib/src/graphql/mod.rs Show resolved Hide resolved
@deekerno
Copy link
Contributor Author

@deekerno

* Left some feedback

* I forget, were there supposed to be insta snapshot tests in this PR as well?

@ra0x3 Working on your suggestions now. Re: insta snapshot tests -- not in this PR, but the next one in which introspection data will be added.

@deekerno deekerno force-pushed the deekerno/add-transparent-pagination-types-to-schema branch from 5ddcaae to bacecbc Compare October 17, 2023 14:49
@deekerno deekerno requested review from lostman and ra0x3 October 17, 2023 15:19
Copy link
Contributor

@ra0x3 ra0x3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @deekerno

@deekerno deekerno merged commit a9c3831 into develop Oct 17, 2023
18 checks passed
@deekerno deekerno deleted the deekerno/add-transparent-pagination-types-to-schema branch October 17, 2023 15:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants