Skip to content

Fix order of last activities#9756

Merged
andreslucena merged 1 commit intodecidim:developfrom
mainio:fix/last-activities-order
Sep 13, 2022
Merged

Fix order of last activities#9756
andreslucena merged 1 commit intodecidim:developfrom
mainio:fix/last-activities-order

Conversation

@ahukkanen
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Noticed that the last activities page is not ordering the activities correctly as it used to in 0.26 because it is missing the default order from the search query. This was broken at #8748.

📌 Related Issues

Testing

  • Go to /last_activities
  • See that items are ordered correctly (newest first)

@ahukkanen ahukkanen added module: core type: fix PRs that implement a fix for a bug labels Aug 18, 2022
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked it locally, it works as expected. Thanks for the PR!

@andreslucena andreslucena merged commit 7287da4 into decidim:develop Sep 13, 2022
@ahukkanen ahukkanen deleted the fix/last-activities-order branch September 13, 2022 09:19
entantoencuanto added a commit that referenced this pull request Sep 13, 2022
* develop:
  Add missing character on code block (#9798)
  Fix hidden error messages on the registration form (#9625)
  Add documentation about configuring ActiveStorage / dynamic file uploads (#9777)
  Add documentation section about customizing cells (#9622)
  Fix hashtags not recognized at the beginning of the string (#9616)
  Fix version pages showing a HTTP 500 error when the version does not exist (#9615)
  Fix multitenant organizations stats cache (#9605)
  Prevent the account edit route through Devise (#9611)
  Fix iframe disabling producing invalid HTML (#9685)
  Fix import of images on spaces (#9779)
  Fix order of last activities (#9756)
  Fix leaking emails on admin user search controller (#9791)
  Ignore participatory spaces without models in meetings visible_for scope (#9790)
eliegaboriau pushed a commit to eliegaboriau/decidim that referenced this pull request Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants