Skip to content
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

Backport 'Allow apps to configure the document types in the verifications module' to v0.27 #12451

Conversation

alecslupu
Copy link
Contributor

🎩 What? Why?

Backport #12306 to v0.27

♥️ Thank you!

* Update i18n translation to id number

* Changing DNI strings to Identification number instead

* Changing NIE string value to Identification number

* Offline review one spec failing

* Local tests working omn id_document_offline & online

* Generators updated initializer and yml secrets. Update to test to fix CLI

* Update to secrets.yml file

* Update to README file in verifications

* Lint code fix to README file

* Change to secrets.yaml file

* Reverting secrets file

* Verifications model change in the secrets file

* Change to config in initalizer.rb

* reverting to previous intializer

* Test to verifications

* Reverting file

* Test with a require engine in verifications: Adapter.rb

* Reverting initalizer file

* Update of the Gemfile in verifications to add a path to verifications

* Update of Gemfile using bundle

* verifiication change in the gemlog file

* Minor change to bundler for gemfile generators on verifations

* Reverting file. Verifications path added to generators Gemfile

* Verification update on generators and generators_spec files to match branch name

* Revert gemfile dependency

* Update of release notes for changes in the PR

* Update to release notes as per suggestion from andres

* Update decidim-generators/lib/decidim/generators/app_templates/initializer.rb

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Reverting RELEASE_NOTES section to previous version

* Reverting initalizer call on verifications

* Update of fix lint errors on RELEASE_NOTES

* Reverting initalizer change to :decidim, :verifications

* Remove to_json as the secret file only translates to array

* Change to catch ID no. in initalizer and secret file

* Changes to value on indentification number

* Changes to form in view

* Reverting file due to error

* Change to Verifications model to align with intializer and secrets file

* Changes to spec for verifications indentification number

* Changes to verifications model to test

* Reverting config accessor verifications

* Changing the generators string to an array

* Change to i18n tasks array

* Final changes to id_document specs in verifications

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Markdownlint changes based on error

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Update decidim-generators/lib/decidim/generators.rb

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

* Update decidim-generators/spec/lib/generators_spec.rb

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

* Update decidim-generators/lib/decidim/generators/app_generator.rb

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

* Update decidim-generators/lib/decidim/generators/app_generator.rb

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

* Update decidim-generators/lib/decidim/generators/app_generator.rb

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

---------

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>
@alecslupu alecslupu added backport Pull Requests that are a backport for a fixed bug module: verifications type: fix PRs that implement a fix for a bug labels Feb 14, 2024
@alecslupu alecslupu added this to the 0.27.6 milestone Feb 14, 2024
Copy link
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

Can you add the notes from the RELEASE_NOTES.md to the CHANGELOG.md 🙏🏽?

@andreslucena
Copy link
Member

Merging with failing check from codecov

@andreslucena andreslucena merged commit 3d89488 into release/0.27-stable Feb 15, 2024
42 of 43 checks passed
@andreslucena andreslucena deleted the backport/0.27/allow-apps-to-configure-the-do-12306 branch February 15, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Pull Requests that are a backport for a fixed bug module: verifications type: fix PRs that implement a fix for a bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants