-
Notifications
You must be signed in to change notification settings - Fork 100
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: Show sidebar toggle in all views #2532
Conversation
d8e3cd4
to
14b460e
Compare
9f641fc
to
84992cf
Compare
This comment was marked as outdated.
This comment was marked as outdated.
d94dfc9
to
40d277e
Compare
This comment was marked as outdated.
This comment was marked as outdated.
6f20813
to
e9b6b31
Compare
Signed-off-by: Christian Hartmann <[email protected]>
Signed-off-by: Christian Hartmann <[email protected]>
e9b6b31
to
425ba29
Compare
/backport to stable4 |
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.
Would it not be easier to to set submissions to an empty array instead?
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.
⬆️ Question still valid, but eitherway fine with it.
They are only set in the frontend, not in the backend. We could do a refactoring as a follow up to move the submissions from this.form to just this.submissions :) |
This fixes #2531 by loading the full form in the results view in beforeMount()
To clean up the code and also remove a double loading of the full form, the check for a linked file has been refactored.
Signed-off-by: Christian Hartmann [email protected]