-
Notifications
You must be signed in to change notification settings - Fork 71
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
React query for form layouts #10140
React query for form layouts #10140
Conversation
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.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
6a9665a
to
d7d824d
Compare
06a8d0d
to
13ebfbe
Compare
13ebfbe
to
ceff8bc
Compare
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.
Veldig bra jobba! Dette var et godt stykke arbeid! 👏 🎖️
frontend/packages/ux-editor/src/features/formDesigner/activeList/activeListSagas.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ux-editor/src/hooks/mutations/useDeleteAppAttachmentMetadataMutation.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ux-editor/src/hooks/mutations/useDeleteAppAttachmentMetadataMutation.ts
Dismissed
Show dismissed
Hide dismissed
frontend/packages/ux-editor/src/hooks/mutations/useUpdateAppAttachmentMetadataMutation.ts
Dismissed
Show dismissed
Hide dismissed
frontend/packages/ux-editor/src/hooks/mutations/useUpdateAppAttachmentMetadataMutation.ts
Dismissed
Show dismissed
Hide dismissed
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.
Great job! 🥳
Description
Added React Query for form layouts, along with some improvements on typing and test coverage. In general, I have moved the logic from
formLayoutSagas
to React Query mutation hooks. I have also added React Query Devtools, which is a very helpful component that is only visible in development mode.In addition, I have changed the logic that sets the "base container" ID. The old code set the ID to some random value, which led to bugs because this could happen multiple times and there was no intuitive way to find it in the list of containers. Therefore I have given it a constant value. Although this might be a better solution, it is still a quick fix, so we should consider finding a better way to handle this.
Related Issue(s)
Verification