Skip to content

Filter participants admin#8104

Merged
leio10 merged 1 commit intodecidim:developfrom
i-need-another-coffee:ale-add-participants-admin
Jun 14, 2021
Merged

Filter participants admin#8104
leio10 merged 1 commit intodecidim:developfrom
i-need-another-coffee:ale-add-participants-admin

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented Jun 4, 2021

🎩 What? Why?

As an administrator I would like to be able to sort the content of the columns in the "Participants" section (back office) with ransak as it is possible to do in other participatory spaces.

The content can be sorted in ascending or descending order by date or alphabetically. This is useful when there are many administrators.

📌 Related Issues

Link your PR to an issue

Testing

Describe the best way to test or validate your PR.

📋 Checklist

🚨 Please review the guidelines for contributing to this repository.

  • CONSIDER adding a unit test if your PR resolves an issue.
  • ✔️ DO check open PR's to avoid duplicates.
  • ✔️ DO keep pull requests small so they can be easily reviewed.
  • ✔️ DO build locally before pushing.
  • ✔️ DO make sure tests pass.
  • ✔️ DO make sure any new changes are documented in docs/.
  • ✔️ DO add and modify seeds if necessary.
  • ✔️ DO add CHANGELOG upgrade notes if required.
  • ✔️ DO add to GraphQL API if there are new public fields.
  • ✔️ DO add link to MetaDecidim if it's a new feature.
  • AVOID breaking the continuous integration build.
  • AVOID making significant changes to the overall architecture.

📷 Screenshots

Please add screenshots of the changes you're proposing
Description

♥️ Thank you!

@alecslupu alecslupu force-pushed the ale-add-participants-admin branch from ddd6c18 to 23461c5 Compare June 5, 2021 06:46
@alecslupu alecslupu force-pushed the ale-add-participants-admin branch from 23461c5 to ce96866 Compare June 9, 2021 04:21
@alecslupu alecslupu changed the title Ale add participants admin Filter participants admin Jun 10, 2021
@alecslupu alecslupu marked this pull request as ready for review June 10, 2021 21:59
Copy link
Copy Markdown
Contributor

@leio10 leio10 left a comment

Choose a reason for hiding this comment

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

Good job!

@leio10 leio10 merged commit 3857554 into decidim:develop Jun 14, 2021
entantoencuanto added a commit that referenced this pull request Jun 29, 2021
* develop: (47 commits)
  New Crowdin updates (#8150)
  Move the webpacker config override to @decidim/webpacker (#8158)
  Fix admin stylesheet dynamic imports (#8154)
  Fix session timeout conflicting with remember me (#7467)
  Allow to create online meetings without an URL (#8152)
  Fix verification route issues (#8146)
  Fix dont save timeout path to session (#8142)
  Fix access to import CSV results in accountability (#8132)
  Fix user report notification reported user name (#8130)
  Allow users to comment and delete their own comments (#8072)
  New Crowdin updates (#8124)
  Fix webpacker issues (#8136)
  Add comments in participatory space presentation page stats block (#8034)
  Split NPM dependencies to more granular packages (#8121)
  Metric is not shown when value is zero for blocked and reported users (#8117)
  Add missing templates translations (#8133)
  Add missing translation for authorization_modals (#8129)
  Polls in meetings (#8065)
  Fix flaky test on initiatives (#8128)
  Filter participants admin (#8104)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants