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

🐛(frontend) wrong store with invite link #2521

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

AntoLC
Copy link
Contributor

@AntoLC AntoLC commented Nov 30, 2023

Purpose

The API calls were still using the persistent store instead of the local store when using the invite link.

Proposal

We removed these double stores logic and now only use 1 store that have a "withPersistency" option.
The invite links set this option to false so that the store is not persistent.

@AntoLC AntoLC self-assigned this Nov 30, 2023
@AntoLC AntoLC force-pushed the issue/anthony/wrong-jwt-store branch 2 times, most recently from c5b7667 to e96acb7 Compare November 30, 2023 11:32
@AntoLC AntoLC marked this pull request as ready for review November 30, 2023 11:42
@AntoLC AntoLC requested a review from lunika November 30, 2023 11:42
The API calls were still using the persistent store instead of the
local store when using the invite link. We removed these double stores
and now only use 1 store that have a "withPersistency" option.
The invite links set this option to false so that the store is not
persistent.
@AntoLC AntoLC force-pushed the issue/anthony/wrong-jwt-store branch from e96acb7 to 28844e1 Compare November 30, 2023 12:59
@AntoLC AntoLC enabled auto-merge (rebase) November 30, 2023 13:00
@AntoLC AntoLC merged commit 1211d7e into master Nov 30, 2023
33 of 34 checks passed
@AntoLC AntoLC deleted the issue/anthony/wrong-jwt-store branch November 30, 2023 13:12
lunika added a commit that referenced this pull request Nov 30, 2023
Fixed

- Fix invite links on BBB (#2521)
lunika added a commit that referenced this pull request Nov 30, 2023
Fixed

- Fix invite links on BBB (#2521)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants