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

Fix: Show sidebar toggle in all views #2532

Merged
merged 2 commits into from
Jan 31, 2025
Merged

Conversation

Chartman123
Copy link
Collaborator

@Chartman123 Chartman123 commented Jan 28, 2025

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]

@Chartman123 Chartman123 added bug Something isn't working javascript Javascript related ticket 2. developing Work in progress labels Jan 28, 2025
@Chartman123 Chartman123 added this to the 5.0 milestone Jan 28, 2025
@Chartman123 Chartman123 force-pushed the fix/2531-sidebar-toggle branch 3 times, most recently from d8e3cd4 to 14b460e Compare January 29, 2025 18:55
@Chartman123 Chartman123 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jan 29, 2025
@Chartman123 Chartman123 self-assigned this Jan 29, 2025
@Chartman123 Chartman123 marked this pull request as ready for review January 29, 2025 18:56
@Chartman123 Chartman123 force-pushed the fix/2531-sidebar-toggle branch 2 times, most recently from 9f641fc to 84992cf Compare January 29, 2025 19:11
@Chartman123

This comment was marked as outdated.

@Chartman123 Chartman123 force-pushed the fix/2531-sidebar-toggle branch from d94dfc9 to 40d277e Compare January 29, 2025 23:38
@Chartman123

This comment was marked as outdated.

@Chartman123 Chartman123 force-pushed the fix/2531-sidebar-toggle branch from 6f20813 to e9b6b31 Compare January 31, 2025 16:09
@Chartman123 Chartman123 force-pushed the fix/2531-sidebar-toggle branch from e9b6b31 to 425ba29 Compare January 31, 2025 16:14
@Chartman123 Chartman123 requested review from susnux and Koc January 31, 2025 16:16
@Chartman123
Copy link
Collaborator Author

/backport to stable4

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Jan 31, 2025
Copy link
Collaborator

@susnux susnux left a 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?

Copy link
Collaborator

@susnux susnux left a 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.

@Chartman123
Copy link
Collaborator Author

Would it not be easier to to set submissions to an empty array instead?

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 :)

@Chartman123 Chartman123 merged commit 1280b9c into main Jan 31, 2025
47 checks passed
@Chartman123 Chartman123 deleted the fix/2531-sidebar-toggle branch January 31, 2025 17:25
@backportbot backportbot bot removed the backport-request Pending backport by the backport-bot label Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working javascript Javascript related ticket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sidebar toggle not shown when you start in results view
2 participants