Skip to content

Add seeded Trustees#10901

Merged
alecslupu merged 2 commits intodevelopfrom
chore/election-trustees-seeds
May 30, 2023
Merged

Add seeded Trustees#10901
alecslupu merged 2 commits intodevelopfrom
chore/election-trustees-seeds

Conversation

@andreslucena
Copy link
Copy Markdown
Member

@andreslucena andreslucena commented May 24, 2023

🎩 What? Why?

Every time you set up a new installation and you want to try Elections you need to add at least 3 Trustees so it works.

This PR adds three users (admin@example.org, user@example.org, and user2@example.org) as Trustees in all the Participatory Spaces (ie an Assembly), so it's easier/faster to test this out.

Testing

  1. Create a new development_app (or re-do the seeds/database with bin/rails db:drop db:create db:migrate db:seed)
  2. Sign in as admin
  3. Go to any Participatory Space (ie an Assembly)
  4. Go to Trustees section in the sidebar
  5. See that you have 3 Trustees

📷 Screenshots

Screenshot of three trustees

♥️ Thank you!

@andreslucena andreslucena added target: developer-experience module: elections type: internal PRs that aren't necessary to add to the CHANGELOG for implementers labels May 24, 2023
@alecslupu
Copy link
Copy Markdown
Contributor

@andreslucena FYI i have removed from template:

#### :pushpin: Related Issues
*Link your PR to an issue*
- Related to #?
- Fixes #?

@alecslupu alecslupu self-requested a review May 25, 2023 19:57
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

image
LGTM

@alecslupu alecslupu merged commit f5d09fe into develop May 30, 2023
@alecslupu alecslupu deleted the chore/election-trustees-seeds branch May 30, 2023 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: elections target: developer-experience type: internal PRs that aren't necessary to add to the CHANGELOG for implementers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants