Màxim Colls

@NilHomedes fixed what you mentioned here https://github.com/decidim/decidim/pull/13417#pullrequestreview-2345863427 ~Assigning it to maintainers already since the fixes were very small. Thanks!~ UPDATE: Sorry, first assigning it to Contractors review. (looks like I...

> @mllocs have u seen this comment as well ? Yes, I'm waiting for @NilHomedes answer, since he told me to put the map in that position.

> or someone tag me in a private note. @NilHomedes is it possible to mention someone in a proposal' private note?

@NilHomedes I implemented the email notification similar to how PokeCode did [here](https://github.com/openpoke/decidim/pull/23). I'm just wondering if it's the right approach since it doesn't use the notifications system, instead it directly...

@NilHomedes @andreslucena Alright, I'll take that approach. Just one thing to consider. It seems that the notifications system couples *one* notification for *one* resource. In our case, one notification for...

@NilHomedes @andreslucena Reagarding the new field in the Notifications Setting form, which should be the default value? Enabled or Disabled?

@NilHomedes there's an existing flow in place that overlaps with the changes in this issue. When a **user logs in for the first time**, they are redirected to this special...

@furilo You're right, that path is broken right now, as confirming the email counts as the first login and the confirmation path doesn't go through [this logic](https://github.com/decidim/decidim/blob/develop/decidim-core/app/controllers/concerns/decidim/devise_authentication_methods.rb#L11). We could take...

@NilHomedes Regarding this AC: > Given a participant who have completed the registration process > When completes one verification, > Then is redirected again to the funnel and can see...

> Great! Looks good to me. Just one suggestion: Would be great to unable the cards of the granted verifications. Would this take too much time? @NilHomedes what do you...