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

[stable22] fix(core): Add password confirmation requirement for getapppassword #39422

Merged
merged 1 commit into from Jul 17, 2023
Merged

[stable22] fix(core): Add password confirmation requirement for getapppassword #39422

merged 1 commit into from Jul 17, 2023

Conversation

nickvergessen
Copy link
Member

Backport #39416

@nickvergessen
fix(core): Add password confirmation requirement for getapppassword
9fd8a02 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added the 3. to review Waiting for reviews label Jul 17, 2023
@nickvergessen nickvergessen requested review from blizzz and a team July 17, 2023 10:18
@nickvergessen nickvergessen self-assigned this Jul 17, 2023
@nickvergessen nickvergessen requested review from ArtificialOwl and come-nc and removed request for a team July 17, 2023 10:18
@blizzz blizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 17, 2023
@nickvergessen nickvergessen merged commit ad8e648 into stable22 Jul 17, 2023
19 of 22 checks passed
@nickvergessen nickvergessen deleted the backport/39416/stable22 branch July 17, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcelklehr marcelklehr marcelklehr approved these changes

@bigcat88 bigcat88 bigcat88 approved these changes

@blizzz blizzz Awaiting requested review from blizzz

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl was automatically assigned from nextcloud/server-backend

@come-nc come-nc Awaiting requested review from come-nc come-nc was automatically assigned from nextcloud/server-backend

Assignees

@nickvergessen nickvergessen

Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@nickvergessen @marcelklehr @bigcat88 @blizzz