-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
Document how to work locally with Elections/Votings #10870
Conversation
@Crashillo this is a first approach to what we talked yesterday on how to use decidim-elections locally. Give it an eye and let me know what you think |
2a4385c
to
67dd386
Compare
Hi there, @andreslucena, I'm getting stuck testing this at the key ceremony step. I tell you my dev setup:
I don't know what else I need. There no console errors. Looking at the code, the key-ceremony.js starts some way, the disabled attribute is added at the beggining, but it should be removed once setupElection and bindEvents functions are finished. What am I missing? |
Sometimes I think that I'd needed to refresh the browser and try it again. The screenshots are from #10804? (so I can give it a try locally) |
Yes, last week I tried it using that branch but in the following video I'm using Screencast.from.05-06-23.13.30.13.mp4As you can notice, in the key ceremony step, it appears the election En ello, perform action... and nothing. You see that my three trustees are connected at the same time. I understand my browser has already the keys for each user (the jwk file) since I cannot download it again from nowhere. The docker container is running under port 8000 also. Can you try as well in our branches to discard anything introduced by the redesign? Or, perhaps better, we may do again a call to set it up in my dev-env, is that ok? |
@Crashillo I think I know what's happening. There's an UX bug, where you need to "Upload your identification keys" before being able to "Perform action" in the elections. This is visible in 0:15 on your video. So, to fix this issue, you should:
This is explained in the docs:
Let me know if that works, if not then ping me on Telegram so we can have a quick videoconference to review this |
I can confirm that is working as expected in the It's important to follow all the steps in the order that's expected, as if not the ceremony can not work. I'll prepare an screencast so you can see how it should work with the "happy path" in |
Here you have: e-voting_feature-redesign-2023-06-26_15.10.15.mp4I'm not sure that the "Subir claves de elección" (Upload elections keys) needs to be done in this step, probably no as with the "admin" user I didn't need to do it and it let me finish the key ceremony. A couple caveats on how I've done this:
Decidim::User.find(1).update(nickname: "admin")
Decidim::User.find(2).update(nickname: "user1")
Decidim::User.find(3).update(nickname: "user2")
docker-compose rm
docker-compose up
And that's pretty much it, I think I'm not missing anything |
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.
@andreslucena can you sync with latest develop, and also fix the linting issues ?
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. great help while reviewing other tickets 🎉
* feature/redesign: Redesign: replace cells with redesigned versions if present and update references (#11123) Remove duplicated constant Simplify logic Enable specs Fix leaflet fix failing specs Fix issues with overriding maps and loading Leaflet (#11105) Update decidim-proposals/app/views/decidim/proposals/proposals/new.html.erb Update decidim-comments/lib/decidim/comments/comments_helper.rb Update decidim-assemblies/spec/system/filter_assemblies_spec.rb Update decidim-assemblies/spec/system/filter_assemblies_spec.rb Document how to work locally with Elections/Votings (#10870) Fix Admin dashboard disappear if you are in Trustee Zone (#11111) Fix Shakapacker upgrade does not work for existing instances (#10814) Fix for sending welcome emails for new participants (#10991) Fix seeded trustees (#10964)
* Document how to work locally with Elections/Votings * Fix spelling * Add testing instructions and docker-compose configuration * Use "there is" instead of "there's" * Document exception shown if the service is not running * Remove unecessary details from documented exception * Fix markdownlint offenses
🎩 What? Why?
During the Redesign there's the need to work fully with
decidim-elections
, without making any changes in the Bulletin Board service. This PR provides a working docker-compose configuration and also documents a Firefox limitation thatTesting
Follow the instructions, create an Election and do the whole process