Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Emit an event for every disabled 2FA provider during cleanup #21344

Merged
merged 1 commit into from
Aug 13, 2020
Merged

Emit an event for every disabled 2FA provider during cleanup #21344

merged 1 commit into from
Aug 13, 2020

Conversation

ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst added 3. to review Waiting for reviews security pending documentation This pull request needs an associated documentation update labels Jun 9, 2020
@ChristophWurst ChristophWurst added this to the Nextcloud 20 milestone Jun 9, 2020
@ChristophWurst ChristophWurst self-assigned this Jun 9, 2020
rullzer
rullzer approved these changes Jun 9, 2020
View reviewed changes
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it!

@ChristophWurst
Emit an event for every disabled 2FA provider during cleanup
68794eb 
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst force-pushed the fix/twofactor-cleanup-event branch from 8c31018 to 68794eb Compare June 16, 2020 12:38
@MorrisJobke MorrisJobke mentioned this pull request Aug 11, 2020
Merged
57 tasks
@ChristophWurst
Copy link
Member Author

🏓 for review ✌️

@MorrisJobke MorrisJobke merged commit 725fece into master Aug 13, 2020
@MorrisJobke MorrisJobke deleted the fix/twofactor-cleanup-event branch August 13, 2020 05:25
@MorrisJobke MorrisJobke mentioned this pull request Aug 13, 2020
Merged
@MorrisJobke
Copy link
Member

Documentation in nextcloud/documentation#3007

@ChristophWurst ChristophWurst removed the pending documentation This pull request needs an associated documentation update label Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MorrisJobke MorrisJobke MorrisJobke approved these changes

@rullzer rullzer rullzer approved these changes

@simonspa simonspa Awaiting requested review from simonspa

@przybylski przybylski Awaiting requested review from przybylski

@debarshiray debarshiray Awaiting requested review from debarshiray

@kesselb kesselb Awaiting requested review from kesselb

@danjpgriffin danjpgriffin Awaiting requested review from danjpgriffin

@affan-hussain affan-hussain Awaiting requested review from affan-hussain

@javierllorente javierllorente Awaiting requested review from javierllorente

@newhinton newhinton Awaiting requested review from newhinton

@anagromataf anagromataf Awaiting requested review from anagromataf

@cosenal cosenal Awaiting requested review from cosenal

Assignees

@ChristophWurst ChristophWurst

Labels
3. to review Waiting for reviews security
Projects
None yet
Milestone
Nextcloud 20
Development

Successfully merging this pull request may close these issues.
3 participants
@ChristophWurst @MorrisJobke @rullzer