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

fix: current view should be updated on a per user basis instead of workspace #363

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

khorshuheng
Copy link
Contributor

Current view should be set and retrieved on per user id basis, instead of per workspace. Otherwise, a user will be able to view the last visited page for another user, even if the user has no permission to view it.

@khorshuheng khorshuheng requested a review from appflowy January 7, 2025 08:19
@khorshuheng khorshuheng marked this pull request as draft January 7, 2025 08:19
@khorshuheng khorshuheng requested a review from LucasXu0 January 7, 2025 08:19
@khorshuheng khorshuheng force-pushed the per-user-current-view branch from 64d3623 to 1fa7b49 Compare January 28, 2025 02:58
@khorshuheng khorshuheng requested a review from Horusiath January 28, 2025 03:03
@khorshuheng khorshuheng marked this pull request as ready for review January 28, 2025 03:15
@khorshuheng khorshuheng force-pushed the per-user-current-view branch from 1fa7b49 to e0ceddc Compare January 28, 2025 04:32
@khorshuheng khorshuheng force-pushed the per-user-current-view branch from e0ceddc to ebfd45e Compare January 28, 2025 04:34
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