Update open data exports for Debates#13698
Merged
andreslucena merged 21 commits intodecidim:developfrom Dec 9, 2024
Merged
Conversation
13 tasks
andreslucena
requested changes
Dec 2, 2024
Member
|
Also there are some relevant failing sepcs, please check that out too |
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
antopalidi
pushed a commit
that referenced
this pull request
Dec 16, 2024
* updated and created (including specs) added * added followers and the test for them * added translations for debates * normalize the tansalations * followers_count removed as followers already added * test to check closed_at when is false * added comments author type and last comment id * fixed updated_at test * last_id and last_type refined * updated transaltions * renamed the followers count method * added endorsements count and test * endorsements_count updated translation * Update decidim-debates/lib/decidim/debates/debate_serializer.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * updated spec for follows count * logic created to retrieve values if a comment is left on a debate * tests for last comment fields if comments are left on debate and test for no values * removed unused i18n translations * updated translations of last_comment_by ans follows_count * renamed author name and url to reflect user --------- Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎩 What? Why?
Update of columns for the csv_export of the Debates component. This is a part of on going work in Ref: MGOV06
As images have not been added yet, its been left out of the serializer. PR: #13521 has more information.
📌 Related Issues
Testing
run
decidim-debates/spec/lib/decidim/debates/debate_serializer_spec.rb📷 Screenshots