Skip to content

[stable27] fix: Add direct parameter to flow auth v2 - #47864

Merged
juliusknorr merged 1 commit into
stable27from
backport/46534/stable27
Sep 18, 2024
Merged

[stable27] fix: Add direct parameter to flow auth v2#47864
juliusknorr merged 1 commit into
stable27from
backport/46534/stable27

Conversation

@backportbot

@backportbotbackportbotBot commented Sep 10, 2024

Copy link
Copy Markdown

Backport of #46534

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@pabzm

Copy link
Copy Markdown
Member

@ChristophWurst I hope to have fixed it, please have a look.

@ChristophWurst

Copy link
Copy Markdown
Member

Checked the diff and everything looks great! I'm asking Julius for a second 👀

@juliusknorr
juliusknorr marked this pull request as ready for review September 11, 2024 17:17
@pabzm
pabzmforce-pushed the backport/46534/stable27 branch from 5172020 to cf9d485CompareSeptember 12, 2024 06:22
@pabzm

Copy link
Copy Markdown
Member

Can I do something about the CI errors?

@ChristophWurst

Copy link
Copy Markdown
Member

Not sure why no job ran. You might try with another rebase

@pabzm
pabzmforce-pushed the backport/46534/stable27 branch from cf9d485 to b381734CompareSeptember 15, 2024 15:28
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@pabzm
pabzmforce-pushed the backport/46534/stable27 branch from b381734 to 1f15850CompareSeptember 17, 2024 18:37
@pabzm

Copy link
Copy Markdown
Member

I think there's nothing more I can currently do to drive this PR forward? If there is, please let me know.

@susnux

Copy link
Copy Markdown
Contributor

27 is eol so force merge needed

@juliusknorr
juliusknorr merged commit 7d2faa6 into stable27Sep 18, 2024
@juliusknorr
juliusknorr deleted the backport/46534/stable27 branch September 18, 2024 23:16
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@pabzm@ChristophWurst@susnux@juliusknorr