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

Derived context values #164

Merged
merged 3 commits into from
Jan 15, 2025
Merged

Derived context values #164

merged 3 commits into from
Jan 15, 2025

Conversation

captbaritone
Copy link
Owner

@captbaritone captbaritone commented Dec 16, 2024

Working toward a future where Grats can enable a "only pay for what you use" model for client-side GraphQL execution. Combined with #160 this should allow different parts of your app to use different expensive (in terms of setup and in terms of bundle size) context values and then only pay for the context values that are accessed by the queries used in your bundle.

Screenshot 2024-12-15 at 10 20 28 PM

Summary: A sketch of derived contexts as described in #159

Not sure this is how the implemenetaiton should work. Was just focusing on getting things working end to end.

If we go this way, I'd want to focus a bit more on internal architecture as well as error handling.

Test Plan:

ghstack-source-id: 0513dda4e4937a4b1ca9b09e1dd17e68557cd2e1
Pull Request resolved: #161
Copy link

netlify bot commented Dec 16, 2024

Deploy Preview for grats ready!

Name Link
🔨 Latest commit 3318383
🔍 Latest deploy log https://app.netlify.com/sites/grats/deploys/678708d4b04c36000898e6b7
😎 Deploy Preview https://deploy-preview-164--grats.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@captbaritone captbaritone force-pushed the derived-resolvers branch 2 times, most recently from 5702b93 to cb81e8e Compare January 15, 2025 00:36
@captbaritone captbaritone changed the title [WIP] Sketch of derived context Derived context values Jan 15, 2025
@captbaritone captbaritone merged commit b37ef6a into main Jan 15, 2025
17 checks passed
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