NEU-DSG / dailp-encoding

Digital Archive of American Indian Languages Preservation and Perseverance
https://dailp.northeastern.edu
MIT License
20 stars 2 forks source link

Clean up the creation of GraphQL clients in our frontend #387

Closed CharlieMcVicker closed 9 months ago

CharlieMcVicker commented 10 months ago

In light of issues Ellen has had and Naomi has reproduced, it was time to simplify all the code creating GraphQL clients in our codebase. Instead of swapping out clients based on if the user has a token, we now have a smarter AuthExchange that ensures we always include a token for authenticated users (and hit /graphql-edit) and that we do not include tokens for un-authenticated users and WordPress GraphQL requests.

I think there is probably still some refining and renaming to do on this, specifically the serverSideClients could be better named.

netlify[bot] commented 10 months ago

Deploy Preview for dailp ready!

Name Link
Latest commit 0db30433cc3006d749c974552bf0c24ef8780b38
Latest deploy log https://app.netlify.com/sites/dailp/deploys/65b0169524b25b00080bad64
Deploy Preview https://deploy-preview-387--dailp.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.