-
-
Notifications
You must be signed in to change notification settings - Fork 571
Allow the superuser to reinvite users 5125 #5410
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
Allow the superuser to reinvite users 5125 #5410
Conversation
…8' into rework_donation_site_import_#4978
…ank, phone is not mandatory, fixed tests
…8' into rework_donation_site_import_#4978
…8' into rework_donation_site_import_#4978
…inor requested changes
…d_item_categories
|
Hello @cielf, can you, please, take a look at these changes, it seems that invitations are resent correctly this way. |
|
@someykoku I'm having some issues with my environment so it might be some time before I could get to this. |
|
Don't worry, no problem! Keep me up to date
…On Mon, 22 Sept 2025, 19:13 CL Fisher, ***@***.***> wrote:
*cielf* left a comment (rubyforgood/human-essentials#5410)
<#5410 (comment)>
Hrm... I'm seeing something quite wrong with an edge case, but am
suspicious of my environment atm. I'm going to recreate, and get someone
else to recreate. Stay tuned.
—
Reply to this email directly, view it on GitHub
<#5410 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUZY5PEPEJMOFXVGFIDRNWT3UAUZ5AVCNFSM6AAAAACHEXL6XCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGMRQGI3DENBUG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
It took awhile, because I needed to do some regression testing, but I think we're ok as far as functionality goes. |
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.
Looks good to me!
|
@someykoku: Your PR |
Resolves #5125
Description
Type of change
How Has This Been Tested?
Screenshots