Skip to content

Conversation

angarita-dev
Copy link
Contributor

Why?

  • changes provider hierarchy to expose logout to UrqlClient (fixes PLAT-354)
  • changes logout behavior to correctly logout dynamic, accessToken & perform redirect to login (fixes PLAT-1823)
  • changes responsibility of AuthProvider
  • pause initial queries that have authentication required
  • handles correct pending projectId queries for logged in user (url /projects/[projectId]/..., useful when linking from generic contexts (blog)
  • de-couples posthog & identifier, allowing us to set the posthog provider higher (unlink on logout) & identifies the user further below the providers when UrqlClient context is available.
  • changes projectId cookie to lastProjectId

How?

  • Based off analogue PR to previous repository, although with less implications / things changing at the same time.

Tickets?

Contribution checklist?

  • The commit messages are detailed
  • The build command runs locally
  • Assets or static content are linked and stored in the project
  • You've reviewed spelling using a grammar checker
  • For documentation, guides or references, you've tested the commands and steps
  • You've done enough research before writing

Security checklist?

  • Sensitive data has been identified and is being protected properly
  • Injection has been prevented (parameterized queries, no eval or system calls)
  • The Components are escaping output (to prevent XSS)

References?

Optionally, provide references such as links

Preview?

Optionally, provide the preview url here

Copy link

changeset-bot bot commented Dec 2, 2024

⚠️ No Changeset found

Latest commit: 289b7b3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@punkbit punkbit force-pushed the develop branch 2 times, most recently from be96e6a to f13d1c5 Compare February 20, 2025 15:39
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.

2 participants