-
-
Notifications
You must be signed in to change notification settings - Fork 398
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
Notification to initiative authors / promotor committee #7028
Notification to initiative authors / promotor committee #7028
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.
Hi @armandfardeau ! I checked in local, and it seems HTML anchors are not accepted for email_intro and email_outro, so I re-wrote the translation keys accordingly. Sorry for the mistake :)
Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com>
This works perfectly, thanks @armandfardeau! |
@armandfardeau watch out, the lint job is failing! |
@carolromero could you confirm this can be merged? |
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.
I have some small details, could you accept them 🙏🏽?
Hi @paulinebessoles just chiming in to confirm this can be merged once the tiny changes suggested by Andrés are accepted. Thanks for the PR @armandfardeau |
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
@mrcasals Hi, everything is in order here, tell me if you need 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.
Nice, thanks!
* Backport decidim#7028 * Normalize locales * Add missing fr translation * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Update decidim-initiatives/config/locales/fr.yml Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com> * Fix malformed yaml * Fix en offenses Co-authored-by: Pauline Bessoles <52420208+paulinebessoles@users.noreply.github.com>
🎩 What? Why?
The author is not notified when a user tries to join the promoter committee
The candidate to the promoting committee is not notified when its application is accepted or rejected.
This results in both parties not managing the initiative as its specification is intended.
As an author, I want to receive a notification when a user applies to join the promoting committee of my initiative
As a promoting committee candidate I want to receive a notification when my application is accepted
As a promoting committee candidate I want to receive a notification when my application is rejected
📌 Related Issues
Link your PR to an issue
Testing
Describe the best way to test or validate your PR.
📋 Checklist
🚨 Please review the guidelines for contributing to this repository.
docs/
.