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

Move locale field from WorkspaceMember to UserWorkspace #9839

Closed
FelixMalfait opened this issue Jan 24, 2025 · 1 comment
Closed

Move locale field from WorkspaceMember to UserWorkspace #9839

FelixMalfait opened this issue Jan 24, 2025 · 1 comment
Assignees

Comments

@FelixMalfait
Copy link
Member

The preferred local field should be deleted on WorkspaceMember.
We should create a new one on UserWorkspace instead. Because we need to local information at a pretty high level / in a metadata or api context, and we don't want to have to go into the workspace schema for that.

@FelixMalfait
Copy link
Member Author

OK we can wait for this, in the end I opted for a pattern where the locale is passed as a header to the API and not retrieved from the logged-in user, which makes this issue less urgent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

No branches or pull requests

1 participant