Skip to content

Voting metrics cell for the landing page#7469

Merged
mrcasals merged 2 commits intofeat/votings_public_landing_pagefrom
feat/metrics_for_public_voting_landing_page
Mar 3, 2021
Merged

Voting metrics cell for the landing page#7469
mrcasals merged 2 commits intofeat/votings_public_landing_pagefrom
feat/metrics_for_public_voting_landing_page

Conversation

@agustibr
Copy link
Copy Markdown
Contributor

🎩 What? Why?

This PR implements the public side of the Decidim::Votings::ContentBlocks::LandingPage::MetricsCell.

Currently there are no metrics implemented for the votings space so this cell is just a placeholder for the metrics to be defined in the future.

📌 Related Issues

Testing

  • As a participant, visit a voting details page (the metrics cell should be activated on the Landing page admin side):
    Screenshot 2021-02-26 at 16 34 52

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

♥️ Thank you!

@agustibr agustibr added project: e-voting Barcelona City Council contract module: elections labels Feb 26, 2021
@agustibr agustibr self-assigned this Feb 26, 2021
@agustibr agustibr force-pushed the feat/metrics_for_public_voting_landing_page branch from 5442165 to 8fc58db Compare February 26, 2021 16:09
Copy link
Copy Markdown
Contributor

@orlera orlera left a comment

Choose a reason for hiding this comment

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

📂

@agustibr agustibr force-pushed the feat/metrics_for_public_voting_landing_page branch from 8fc58db to f74cfc3 Compare March 1, 2021 10:37
Copy link
Copy Markdown
Contributor

@slickepinne slickepinne left a comment

Choose a reason for hiding this comment

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

👍

def metrics
nil

# @metrics ||= VotingsMetricChartsPresenter.new(participatory_process: current_participatory_space)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this comment needed? 😊

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, it is just a hint for when the metrics are available. 😁

@agustibr agustibr force-pushed the feat/metrics_for_public_voting_landing_page branch from f74cfc3 to f8004ab Compare March 2, 2021 18:10
@agustibr agustibr force-pushed the feat/metrics_for_public_voting_landing_page branch from f8004ab to cfffb3b Compare March 3, 2021 06:54
@mrcasals mrcasals merged commit 88a9bfc into feat/votings_public_landing_page Mar 3, 2021
@mrcasals mrcasals deleted the feat/metrics_for_public_voting_landing_page branch March 3, 2021 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: elections project: e-voting Barcelona City Council contract

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants