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

Feat/add-graph-endpoints #1

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Feat/add-graph-endpoints #1

wants to merge 16 commits into from

Conversation

patroza
Copy link
Member

@patroza patroza commented Feb 7, 2023

Converts some endpoints to a GraphQL like experience:

  • Combine multiple queries into a single request. batch queries.
  • Optionally request follow-up queries to mutations. so within a single roundtrip we can mutate, and query.
  • RPC semantics, so all supported errors are returned within Either Left, instead of encoded to and from HTTP Status Codes + Bodies.

TODOs

  • improve type inference; when requesting result or additional follow-up queries, the response type should no longer have these fields optional.
  • improve ergonomics, unwrap the Eithers

Future

  • select individual fields (concept already done)

@patroza patroza force-pushed the feat/add-graph-endpoints branch from 2a73748 to 9953376 Compare February 7, 2023 08:54
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 9953376 to 7aa8907 Compare February 7, 2023 10:20
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 7aa8907 to 1bb6952 Compare February 7, 2023 12:37
@patroza patroza force-pushed the feat/add-graph-endpoints branch 2 times, most recently from 9846fac to 8ce5bbf Compare February 8, 2023 07:29
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 8ce5bbf to 728be90 Compare February 8, 2023 07:29
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 1c9c602 to 85751cd Compare February 8, 2023 19:58
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 85751cd to 788a91c Compare February 9, 2023 08:22
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 788a91c to e01a9f8 Compare February 9, 2023 08:43
@patroza patroza force-pushed the feat/add-graph-endpoints branch from e01a9f8 to 1e19564 Compare February 10, 2023 11:29
@patroza patroza force-pushed the feat/add-graph-endpoints branch 3 times, most recently from 81a4e79 to b405908 Compare February 11, 2023 12:18
@patroza patroza force-pushed the feat/add-graph-endpoints branch from b405908 to 29c443c Compare February 12, 2023 08:08
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 29c443c to 6f55106 Compare February 15, 2023 08:46
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 6f55106 to 27de840 Compare February 17, 2023 12:22
@patroza patroza force-pushed the feat/add-graph-endpoints branch 2 times, most recently from b036c72 to f92f6de Compare April 24, 2023 06:49
@patroza patroza force-pushed the feat/add-graph-endpoints branch from f92f6de to 66f03b0 Compare April 24, 2023 08:24
@patroza patroza force-pushed the feat/add-graph-endpoints branch from 66f03b0 to cd2ad49 Compare April 26, 2023 09:17
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.

1 participant