-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(api): Change views API in the backend #409
Conversation
Change project serialization step to include `views` dictionary rather than a single `layout`.
Remove duplicate test
There are a lot of changes, so I really made an effort to make each commit self-contained. Please review commit-by-commit, and don't hesitate to say if a commit should be delayed/put in a separate PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting for your feedbacks 🙏🏻 .
This PR is an API breaking-change, and will break the frontend in state.
Do you think @rouk1 you can work inside this PR as a full-feature PR? If not, could you branch from it and merge on it?
What do you think?
e67b289
to
d5be28f
Compare
d5be28f
to
943c50b
Compare
This PR should be trashed. We chose with @augustebaum to work in a full feature PR that will come from the |
Closed in profit of #414 |
key
argumentLayoutItemSize
Addresses #336