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.

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

Fix missing native submit type from OAuth client add button #35186

Merged
merged 1 commit into from
Nov 17, 2022

Conversation

ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented Nov 15, 2022

Untested because I'm unable to build server atm.

Regression of #32521. An ordinary button doesn't make a form submit. A native submit button is required for that.

Fixes #35045

@ChristophWurst
Copy link
Member Author

/backport to stable25

@ChristophWurst
Copy link
Member Author

/compile amend /

@giteichi
Copy link

Could not test this fix with my running server, but adding type="submit" to the button via browser debugger fixes the problem

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 16, 2022
@ChristophWurst
Copy link
Member Author

Tried with the compiled assets. The button doesn't have the native type.

@ChristophWurst ChristophWurst added 2. developing Work in progress and removed 4. to release Ready to be released and/or waiting for tests to finish labels Nov 17, 2022
@ChristophWurst ChristophWurst marked this pull request as draft November 17, 2022 09:14
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@ChristophWurst ChristophWurst force-pushed the fix/oauth2-add-client-button-native-submit branch from 9d07a66 to ab8b6cf Compare November 17, 2022 09:55
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 17, 2022
@ChristophWurst ChristophWurst marked this pull request as ready for review November 17, 2022 09:55
@ChristophWurst
Copy link
Member Author

Works now. I accidentally passed a reference to submit instead of the string 'submit'.

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 17, 2022
@ChristophWurst ChristophWurst merged commit 183487a into master Nov 17, 2022
@ChristophWurst ChristophWurst deleted the fix/oauth2-add-client-button-native-submit branch November 17, 2022 17:23
@ckvtz
Copy link

ckvtz commented Feb 7, 2023

Hi ChristophWurst,

I am facing a similar issue trying to integrate openproject with my nextcloud community version 25.0.3.

Has this issue been fixed in this version? Can you help if there is any workaround?

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug regression
Projects
Development

Successfully merging this pull request may close these issues.

[Bug]: can't add oauth2 client
5 participants