Skip to content

Allow apps to configure the document types in the elections module#11986

Merged
alecslupu merged 5 commits intodevelopfrom
feature/election-document-types
Nov 13, 2023
Merged

Allow apps to configure the document types in the elections module#11986
alecslupu merged 5 commits intodevelopfrom
feature/election-document-types

Conversation

@andreslucena
Copy link
Copy Markdown
Member

@andreslucena andreslucena commented Nov 10, 2023

🎩 What? Why?

The elections module has a feature in the Census voting that talks about DNI and NIE, some document types specific from Spain.

This PR changes it to "Identification number" and also allows applications to configure whatever document types they need.

The idea is to implement this first for decidim-elections and then make a pretty similar change for #9297

📌 Related Issues

Testing

Everthing should be green
Following the documentation should allow you to change these values

♥️ Thank you!

github-actions[bot]
github-actions bot previously approved these changes Nov 10, 2023
github-actions[bot]
github-actions bot previously approved these changes Nov 11, 2023
@andreslucena andreslucena marked this pull request as ready for review November 12, 2023 14:56
@andreslucena
Copy link
Copy Markdown
Member Author

The idea is to have something that will be pretty much the same for #9297. The main difference is that as the elections module is still under development (i.e. not officially "published" in a Releases Notes) then we don't need to add the instructions in the CHANGELOG.

In the case of decidim-verifications we will need to add the upgrade notes as it's a module actively used by the community.

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.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

Elections talk about DNI/NIE and that's only for Spain

2 participants