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.

Sign up for GitHub

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

Jump to bottom

fix: use `decidim_sanitize_admin` instead of `decidim_escape_translated` #626

Merged

ayuki-joto merged 1 commit into main from fix-escape-accountability-timeline

Aug 6, 2024

Collaborator

takahashim commented Aug 5, 2024 •

edited

Loading

🎩 What? Why?

Fix invalid escaping(?) in timeline in accountability module.

📌 Related Issues

Related to Backport 'Patch components and spaces factories' to v0.27 decidim/decidim#12547

📋 Subtasks

Add CHANGELOG upgrade notes, if required
If there's a new public field, add it to GraphQL API
Add documentation regarding the feature
Add/modify seeds
Add tests
Another subtask

📷 Screenshots (optional)

Before

before

After

after


          fix: use decidim_sanitize_admin instead of decidim_escape_translated

863bcb7

takahashim marked this pull request as ready for review

August 5, 2024 11:39

ayuki-joto merged commit 47d3393 into main

2 checks passed

takahashim deleted the fix-escape-accountability-timeline branch

August 6, 2024 06:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet