-
-
Notifications
You must be signed in to change notification settings - Fork 395
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 timeline entries display in result page #13033
Conversation
109a9ad
to
d33581a
Compare
In a meeting with @decidim/product we mentioned that we need to actually show the result children too in this page. I'll mark this as draft until I fix that |
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.
👍
Adding the |
* Fix timeline entries display in result page * Fix rspec description (copy and paste error) * Prevent XSS in timeline entries * Add spec * Show subresults in result page * Link back to parent result on subresults page * Prevent showing an empty element in the breadcrumb on the parent result page
🎩 What? Why?
With the redesign we have removed the Timeline entries from a Result page in Accountability. This PR fixes it.
Although the code was showing children results as timeline entries, these are different things. Also, to clarify, as a Subresult can't have more children results, it doesn't make sense to show them in this page.After diving in Legacy and seeing how the Ancient people wanted this to work (and specially talking with @carolromero a lot about this feature and this page), we have the following result:
Testing
📷 Screenshots