- Issue created by @mariiadeny
- Status changed to Needs review
8 months ago 3:08pm 19 August 2024 - 🇩🇪Germany geek-merlin Freiburg, Germany
+1 for this, it makes the module more useful.
Code review: Looks solid, after all it's just moving code around and making it pluggable.
Did not test it (and the module has not moved its tests to gitlab ci).
@mariia: You may want to learn adding MRs, it has lots of benefits (maintainers often like it, gitlab ci, and more :-).
- First commit to issue fork.
- 🇧🇪Belgium lobsterr
I have updated the code with latest changes from RC branch, plus renamed the classes to keep the same naming everywhere
- Merge request !43Issue #3468964: Introduce group invitation handler, add admin permission. → (Merged) created by lobsterr
- Merge request !44Issue #3468964: Introduce group invitation handler service, add admin permission. → (Merged) created by lobsterr
-
lobsterr →
committed 9e2650fa on 2.x
Issue #3468964: Move sending invitation email from...
-
lobsterr →
committed 9e2650fa on 2.x
-
lobsterr →
committed bd1ce04b on 4.0.x
Issue #3468964: Move sending invitation email from...
-
lobsterr →
committed bd1ce04b on 4.0.x
-
lobsterr →
committed 433a3dd8 on 3.0.x
Issue #3468964: Move sending invitation email from...
-
lobsterr →
committed 433a3dd8 on 3.0.x
Automatically closed - issue fixed for 2 weeks with no activity.