Skip to content

Create Votings participatory space#7145

Merged
mrcasals merged 19 commits intodevelopfrom
feat/add_votings_space
Jan 22, 2021
Merged

Create Votings participatory space#7145
mrcasals merged 19 commits intodevelopfrom
feat/add_votings_space

Conversation

@agustibr
Copy link
Copy Markdown
Contributor

@agustibr agustibr commented Jan 14, 2021

🎩 What? Why?

This PR creates a new participatory space named Votings (inside the decidim-elections module)

Admin users can:

  • Create new votings from the admin panel
  • See the list of votings from the admin panel
  • View votings events in Admin activity Logs

A voting is:

  • Publicable
  • Filterable
  • Sortable by creation date and randomly

NB: some portions of code are commented out and will be addressed in an upcoming PR (#7160 - Manage Votings)

📌 Related Issues

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

add

♥️ Thank you!

@agustibr agustibr force-pushed the feat/add_votings_space branch 4 times, most recently from da43e2a to a0229fb Compare January 14, 2021 08:22
@agustibr agustibr self-assigned this Jan 14, 2021
@agustibr agustibr changed the title Create voting spaces Create Votings participatory space Jan 14, 2021
@agustibr agustibr added space: votings project: e-voting Barcelona City Council contract and removed module: elections labels Jan 14, 2021
@agustibr agustibr force-pushed the feat/add_votings_space branch 9 times, most recently from a57f79b to 4c4c031 Compare January 16, 2021 16:56
slickepinne
slickepinne previously approved these changes Jan 18, 2021
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.

Looks great! 😊

@orlera orlera force-pushed the feat/add_votings_space branch from c065398 to beaf8ae Compare January 18, 2021 15:58
@orlera orlera mentioned this pull request Jan 18, 2021
2 tasks
Copy link
Copy Markdown
Contributor

@beagleknight beagleknight left a comment

Choose a reason for hiding this comment

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

Nice!

@orlera
Copy link
Copy Markdown
Contributor

orlera commented Jan 19, 2021

@decidim/mantainers this has been approved by product 🎉
Could you please review it? 🙂

@orlera orlera mentioned this pull request Jan 19, 2021
12 tasks
validates :end_time, presence: true, date: { after: :start_time }
validates :banner_image, passthru: { to: Decidim::Votings::Voting }
validates :introductory_image, passthru: { to: Decidim::Votings::Voting }
# validate :notify_missing_attachment_if_errored
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.

This method is defined but it's not being applied, not sure if you'll handle this in another PR @agustibr @orlera

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.

Thank you @mrcasals! Yes, it is already handled in an upcoming PR 👍

@mrcasals mrcasals merged commit 0ba47c5 into develop Jan 22, 2021
@mrcasals mrcasals deleted the feat/add_votings_space branch January 22, 2021 10:53
@mrcasals
Copy link
Copy Markdown
Contributor

Good job! @agustibr @orlera I left a small comment but I'm sure you'll handle it in future PRs, so merged!

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.

Create a voting

6 participants