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

Stop survey creator page from crashing on load #3727

Merged

Conversation

ivarnakken
Copy link
Member

Description

The error object was not a valid child of the span. Since it did not work as intended and I do not see the value in fixing it, I simply removed it.

Also add a navigation tab back to the survey list, as it was much needed.

Testing

  • I have thoroughly tested my changes.

It now does not crash.

The `error` object was not a valid child of the span. Since it did
not work as intended and I do not see the value in fixing it, I
simply removed it.

Also add a navigation tab back to the survey list, as it was much
needed.
@ivarnakken ivarnakken added priority:high Pull requests that have high priority, and should therefore be prioritized review-needed Pull requests that need review bug-fix Pull requests that fix a bug labels Mar 29, 2023
@ivarnakken ivarnakken requested a review from a team March 29, 2023 16:11
@ivarnakken ivarnakken self-assigned this Mar 29, 2023
@linear
Copy link

linear bot commented Mar 29, 2023

@LudvigHz LudvigHz added approved Pull requests that have been approved ready-to-merge Pull requests that have been approved and are ready to be merged and removed review-needed Pull requests that need review labels Mar 29, 2023
@ivarnakken ivarnakken merged commit 0336408 into master Mar 29, 2023
@ivarnakken ivarnakken deleted the ivarnakken/aba-354-not-possible-to-add-a-new-survey branch March 29, 2023 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug priority:high Pull requests that have high priority, and should therefore be prioritized ready-to-merge Pull requests that have been approved and are ready to be merged
Projects
None yet
3 participants