-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
Add and use decidim-bulletin_board gem #6997
Add and use decidim-bulletin_board gem #6997
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is so clean ✨
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done 😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is soo good! 👏 🔝
@@ -66,6 +66,7 @@ def election_data | |||
{ | |||
iat: Time.now.to_i, | |||
election_id: election_id, | |||
message_id: "#{election_id}.create_election+a.#{bulletin_board.authority_slug}", | |||
type: "create_election", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can remove the election_id
and create_election
, as they are included in the message_id
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I have that in mind. I wanted to do that with the refactoring of moving the whole election_data
to the gem.
ccf8a77
dec1d9d
to
ccf8a77
Compare
@decidim/core Could you review this PR, please? 🙏 |
ccf8a77
to
d4d51ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
🎩 What? Why?
This PR refactors the election module. Since we have a
decidim-bulletin_board
gem now, we removed the bb client logic fromdecidim
.Testing
The
decidim-bulletin_board
is a dependency for theelections module
. If you setup an election for example, this new gem gets used. There's also a system and unit test.📋 Checklist
🚨 Please review the guidelines for contributing to this repository.
docs/
.📷 Screenshots
Please add screenshots of the changes you're proposing