Skip to content

Fix email verification - #30491

Merged
blizzz merged 1 commit into
masterfrom
fix/30374/email-verification
Jan 7, 2022
Merged

Fix email verification#30491
blizzz merged 1 commit into
masterfrom
fix/30374/email-verification

Conversation

@Pytal

@PytalPytal commented Jan 5, 2022

Copy link
Copy Markdown
Member

@PytalPytal added this to the Nextcloud 24 milestone Jan 5, 2022
@Pytal
Pytal requested a review from a teamJanuary 5, 2022 04:12
@PytalPytal self-assigned this Jan 5, 2022
@Pytal
Pytal requested review from PVince81, come-nc and skjnldsv and removed request for a teamJanuary 5, 2022 04:12
@Pytal

Pytal commented Jan 5, 2022

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@Pytal

Pytal commented Jan 5, 2022

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@blizzz

Copy link
Copy Markdown
Member

can we have a unit test for this?

@Pytal
Pytalforce-pushed the fix/30374/email-verification branch 6 times, most recently from a9c26ae to da95465CompareJanuary 5, 2022 23:57
@Pytal
Pytalforce-pushed the fix/30374/email-verification branch 2 times, most recently from 4b425b3 to 0fdba2cCompareJanuary 6, 2022 20:03
Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal
Pytalforce-pushed the fix/30374/email-verification branch from 0fdba2c to 19a3656CompareJanuary 6, 2022 23:47
@Pytal

Pytal commented Jan 7, 2022

Copy link
Copy Markdown
MemberAuthor

Test updated and passing, would appreciate a re-review or another review now with the updated test :)

@blizzz
blizzz merged commit e65431d into masterJan 7, 2022
@blizzz
blizzz deleted the fix/30374/email-verification branch January 7, 2022 10:46
@PytalPytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 7, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: emailsfeature: users and groups

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VerifyUserData job always gets readded

4 participants

@Pytal@blizzz@skjnldsv@come-nc