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

supply missing fields argument #2158

Merged
merged 1 commit into from
Jan 12, 2022
Merged

supply missing fields argument #2158

merged 1 commit into from
Jan 12, 2022

Conversation

ghostwheel42
Copy link
Contributor

What type of PR?

bug-fix

What does this PR do?

adds missing argument "fields" to login form.

Related issue(s)

Prerequisites

Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

  • In case of feature or enhancement: documentation updated accordingly
  • Unless it's docs or a minor change: add changelog entry file.

@mergify
Copy link
Contributor

mergify bot commented Jan 11, 2022

Thanks for submitting this pull request.
Bors-ng will now build test images. When it succeeds, we will continue to review and test your PR.

bors try

Note: if this build fails, read this.

bors bot added a commit that referenced this pull request Jan 11, 2022
@bors
Copy link
Contributor

bors bot commented Jan 11, 2022

try

Build succeeded:

@ghostwheel42 ghostwheel42 added priority/p1 (Critical) bug with workaround / Should have type/backport Automatic backport this PR to the current stable release type/bug Bug. Not working as intended labels Jan 11, 2022
@ghostwheel42 ghostwheel42 requested a review from Diman0 January 11, 2022 20:54
Copy link
Member

@Diman0 Diman0 left a comment

Choose a reason for hiding this comment

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

lgtm

@mergify
Copy link
Contributor

mergify bot commented Jan 12, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Jan 12, 2022

Build succeeded:

@bors bors bot merged commit e5aa48a into Mailu:master Jan 12, 2022
bors bot added a commit that referenced this pull request Jan 12, 2022
2159: supply missing fields argument (backport #2158) r=mergify[bot] a=mergify[bot]

This is an automatic backport of pull request #2158 done by [Mergify](https://mergify.com).


---


<details>
<summary>Mergify commands and options</summary>

<br />

More conditions and actions can be found in the [documentation](https://docs.mergify.com/).

You can also trigger Mergify actions by commenting on this pull request:

- ``@Mergifyio` refresh` will re-evaluate the rules
- ``@Mergifyio` rebase` will rebase this PR on its base branch
- ``@Mergifyio` update` will merge the base branch into this PR
- ``@Mergifyio` backport <destination>` will backport this PR on `<destination>` branch

Additionally, on Mergify [dashboard](https://dashboard.mergify.com/) you can:

- look at your merge queues
- generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com
</details>


Co-authored-by: Alexander Graf <ghostwheel42@users.noreply.github.com>
@ghostwheel42 ghostwheel42 deleted the fix_sso branch January 13, 2022 15:35
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority/p1 (Critical) bug with workaround / Should have type/backport Automatic backport this PR to the current stable release type/bug Bug. Not working as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Login attempt on roundcube triggers error 500 on /sso/login endpoint (ZeroDivisionError: division by zero)
2 participants