Sessions #507
Merged
Sessions #507
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
@AlecAivazis is attempting to deploy a commit to the HoudiniGraphQL Team on Vercel. A member of the Team first needs to authorize it. |
4 tasks
AlecAivazis
added a commit
that referenced
this pull request
Sep 4, 2022
* Merge fehnomenal's work (#507) * Allow passing session data to the fetch function * Prefer the client side session * Receive the server session only in the browser * Separate client and server sessions more clearly * make test more resilient * fake root layout file * fake +layout.server.js * kit transform threads session from +layout.server.js to client * remove client.init Co-authored-by: Andreas Fehn <[email protected]> * rename setServerSession to setSession * receiving server session data needs to be reactive * document no more init * update authentication guide * auth guide tweaks * use App.Session for SessionData * remove document proxy * added integration test for session * make checker happy * add changeset * remove unused import * fix inconsistent absolutely path in CI * tweak util names * find_exported_fn considers export const * better support for load functions with implicit returns * track hydrated state in root layout * pass session to client side operations (mutations, loadNextPage, etc) * updated snapshots * add log if no session in locals * block on error * document onError blocking * update snapshot * fix duplicate test name Co-authored-by: Andreas Fehn <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is just to bring @fehnomenal's changes into a branch local to houdini