Skip to content

Display meetings count in directory page#7972

Merged
mrcasals merged 1 commit intodevelopfrom
feat/show_meeting_counter
May 10, 2021
Merged

Display meetings count in directory page#7972
mrcasals merged 1 commit intodevelopfrom
feat/show_meeting_counter

Conversation

@ferblape
Copy link
Copy Markdown
Contributor

@ferblape ferblape commented May 9, 2021 •
edited
Loading

🎩 What? Why?

This PR adds the number of total meetings visible in global /meetings page

📌 Related Issues

Testing

  • Go to /meetings and check the number of meetings is visible
  • Play with filters and see how this number changes, and should match with the number of listed meetings

📋 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

List with future events:

Screenshot from 2021-05-10 05-32-51

List with past events:

Screenshot from 2021-05-10 05-32-57

♥️ Thank you!

@ferblape ferblape linked an issue May 9, 2021 that may be closed by this pull request
Add meeting counter with the number of meetings #7725
Closed
2 tasks
@ferblape ferblape requested a review from mrcasals May 10, 2021 03:34
@ferblape ferblape marked this pull request as ready for review May 10, 2021 03:34
@ferblape
Copy link
Copy Markdown
Contributor Author

ferblape commented May 10, 2021

@decidim/product please check in https://decidim.populate.tools/meetings

@carolromero
Copy link
Copy Markdown
Member

carolromero commented May 10, 2021

@ferblape working great, sending to technical review!

@mrcasals mrcasals merged commit 6ac3b64 into develop May 10, 2021
@mrcasals mrcasals deleted the feat/show_meeting_counter branch May 10, 2021 13:30
entantoencuanto added a commit that referenced this pull request May 12, 2021
@entantoencuanto
Merge branch 'develop' into 7285-meetings-merge-minutes-and-close-act…
7e77c42 
…ions

* develop: (23 commits)
  New Crowdin updates (#8001)
  Fix editor when formatting starts with a linebreak (#7999)
  Polling station officer ballot count (#7823)
  New Crowdin updates (#7995)
  Election log (#7923)
  New Crowdin updates (#7988)
  Fixing failing tests on develop branch (#7991)
  Add notes to changelog (#7987)
  Add 2 additional queues for a better scalling of the application (#7986)
  Add announcements to assemblies (#7971)
  Display meetings count in directory page (#7972)
  Touch the reportable object when is hidden to reset caches (#7966)
  Hide moderated meetings and proposals from admin lists (#7974)
  Hide moderated proposals from comparator (#7975)
  Add comment count to the proposal cache (#7965)
  Simplify SQL query for endorsement stats (#7973)
  Show confirmation modal when leaving a meeting (#7970)
  Fix: #7889 - Each comment in a meeting generates a new version of the meeting (#7978)
  Add cache layer for ActivityCell boxes (#7967)
  New Crowdin updates (#7980)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mrcasals mrcasals Awaiting requested review from mrcasals

Assignees

No one assigned

Labels

module: meetings

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add meeting counter with the number of meetings

3 participants

@ferblape @carolromero @mrcasals