Skip to content

Fix handling of non-object troubleshooting data - #7826

Open
vichekaoeun wants to merge 1 commit into
mozilla:mainfrom
vichekaoeun:fix-troubleshoot-validation
Open

Fix handling of non-object troubleshooting data#7826
vichekaoeun wants to merge 1 commit into
mozilla:mainfrom
vichekaoeun:fix-troubleshoot-validation

Conversation

@vichekaoeun

Copy link
Copy Markdown
Contributor

Resolved bug #3116 causing 500 error when troubleshooting data contains JSON that is not an object. Added tests for non-object JSON on form

@denyshon

Copy link
Copy Markdown
Contributor

Should we also add a check for this to clean_content() and display an error if the data is not a valid JSON/dict?

@vichekaoeun

Copy link
Copy Markdown
Contributor Author

yes I think that would make sense but I think it could go into a separate function like clean_troubleshooting since it would mix unrelated issues to the wrong field otherwise.

@vichekaoeun
vichekaoeun force-pushed the fix-troubleshoot-validation branch from e4f0f63 to ce4ca5b Compare August 13, 2026 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants