Skip to content

Upgrade to decidim-bulletin_board 0.15.2#7659

Merged
mrcasals merged 4 commits intodevelopfrom
chore/upgrade-decidim-bulletin-board-to-0.15.2
Mar 24, 2021
Merged

Upgrade to decidim-bulletin_board 0.15.2#7659
mrcasals merged 4 commits intodevelopfrom
chore/upgrade-decidim-bulletin-board-to-0.15.2

Conversation

@beagleknight
Copy link
Copy Markdown
Contributor

@beagleknight beagleknight commented Mar 22, 2021

🎩 What? Why?

This upgrades decidim-elections to use the latest version of the released gems related to the bulletin board.

As mentioned in the code both voting_schemes-dummy and voting_schemes-electionguard will be moved outside of the decidim codebase in a future release.

leio10
leio10 previously approved these changes Mar 22, 2021
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.

Another awesome work!! 🔝 👏


raise StandardError, "Wrong status for the election with published results" if bb_election.status != "results_published"
bulletin_board.publish_results(election.id) do |message_id|
create_election_action(message_id)
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.

Thanks for taking care of this! ❤️ 🤩

orlera
orlera previously approved these changes Mar 22, 2021
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.

💯

slickepinne
slickepinne previously approved these changes Mar 22, 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.

Nice! 🌻

agustibr
agustibr previously approved these changes Mar 23, 2021
Copy link
Copy Markdown
Contributor

@agustibr agustibr left a comment

Choose a reason for hiding this comment

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

🚀🚀

@leio10 leio10 dismissed stale reviews from agustibr, slickepinne, orlera, and themself via 158a700 March 23, 2021 08:52
@leio10 leio10 added project: e-voting Barcelona City Council contract module: elections labels Mar 23, 2021
@mrcasals
Copy link
Copy Markdown
Contributor

@beagleknight tests are failing!

@leio10
Copy link
Copy Markdown
Contributor

leio10 commented Mar 24, 2021

@mrcasals tests are passing now, so this is ready to be reviewed! 💪

@mrcasals mrcasals merged commit d251955 into develop Mar 24, 2021
@mrcasals mrcasals deleted the chore/upgrade-decidim-bulletin-board-to-0.15.2 branch March 24, 2021 15:35
entantoencuanto added a commit that referenced this pull request Mar 26, 2021
* develop: (64 commits)
  Fix report mailers when author is a meeting (#7683)
  New Crowdin updates (#7729)
  Fix form builder assuming proposals module availability (#7689)
  Fix a series of issues with proposal attachments in the public area (#7699)
  New Crowdin updates (#7711)
  Add accessibility labels to the <nav> menus (#7709)
  Fix heading order on the home page (#7710)
  Fix dropdown menu accessibility audits (#7708)
  Fix the aria attribute names (no aria prefix) (#7707)
  Fix validations for registration related fields in Conference form (#7675)
  New Crowdin updates (#7613)
  Use comments counter cache instead of additional query (#7627)
  Upgrade to decidim-bulletin_board 0.15.2 (#7659)
  Bump mimemagic to 0.3.6 (#7701)
  Ensure pagination elements per page is a valid option (#7680)
  Fix link to "Getting started guide" in README.adoc (#7695)
  Fix link to CONTRIBUTING.adoc in PR template (#7696)
  Fix the screen reader class name for comments opinion toggle (#7698)
  Fix initiative-m card hashtags (#7679)
  Don't run all jobs on every PR (#7693)
  ...
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.

6 participants