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

[Workspace] remove the WorkspaceOptions entity #2254

Open
Elorfin opened this issue Oct 20, 2022 · 0 comments
Open

[Workspace] remove the WorkspaceOptions entity #2254

Elorfin opened this issue Oct 20, 2022 · 0 comments

Comments

@Elorfin
Copy link
Member

Elorfin commented Oct 20, 2022

Q A
Bug report? no
Version? all

The Workspace entity has a One-to-One relationship with a WorkspaceOptions entity to store some option in a json encoded format.

  • This has a negative impact on performances (DB queries / serialization of props...).
  • Most of the workspaces options are directly set on the Workspace entity.
  • Poor readability and hard to migrate if some of the embedded options change.

This is an easy pick but I will need to create an updater to migrate the options and this will be an huge one for the platforms.

@Elorfin Elorfin changed the title [Workspace] removes the WorkspaceOptions entity [Workspace] remove the WorkspaceOptions entity Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant