Skip to content
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

Metric is not shown when value is zero for blocked and reported users #8117

Merged
merged 2 commits into from Jun 15, 2021

Conversation

roxanaopr
Copy link
Contributor

🎩 What? Why?

When a given metric has the value zero, then it’s not shown in the Dashboard Metrics section.When no users are blocked and the metric job has started, the "Blocked users" box is not displayed.

📌 Related Issues

Issue : #8116

Testing

Unblock all users
Check metrics from Dashboard
No metric displayed for blocked users

📋 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!

@roxanaopr roxanaopr force-pushed the fix/metric-shown-value-zero branch from f2826af to 21c30b6 Compare June 11, 2021 07:10
@roxanaopr roxanaopr marked this pull request as ready for review June 11, 2021 07:54
@leio10 leio10 added in-review module: admin type: fix PRs that implement a fix for a bug labels Jun 14, 2021
Copy link
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.

LGTM, @decidim/product please confirm that this three metrics should always be shown in the admin zone (blocked users, reported users and user reports).

@carolromero
Copy link
Member

please confirm that this three metrics should always be shown in the admin zone (blocked users, reported users and user reports).

@leio10 confirmed, thanks for the PR @roxanaopr!

@leio10 leio10 merged commit 13c28a0 into decidim:develop Jun 15, 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
Labels
in-review module: admin type: fix PRs that implement a fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants