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

[stable22] make Provisioning API aware of multiple mails #27719

Merged
merged 16 commits into from
Jul 1, 2021

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #27474

blizzz added 15 commits June 30, 2021 05:15
- adding as usual
- deleting and scope setting via additional endpoint

Signed-off-by: Arthur Schiwon <[email protected]>
- also fixes scope of internal methods

Signed-off-by: Arthur Schiwon <[email protected]>
Signed-off-by: Arthur Schiwon <[email protected]>
- format as stored previously in oc_accounts table is kept

Signed-off-by: Arthur Schiwon <[email protected]>
Signed-off-by: Arthur Schiwon <[email protected]>
Signed-off-by: Arthur Schiwon <[email protected]>
- fixes wrong veriable usage also

Signed-off-by: Arthur Schiwon <[email protected]>
Signed-off-by: Arthur Schiwon <[email protected]>
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Waiting for 22.0.1?

@blizzz
Copy link
Member

blizzz commented Jun 30, 2021

Waiting for 22.0.1?

I'd actually get in the RC2. After all the whole bits needs to be backported, if anything breaks earlier is better than later.

@blizzz blizzz requested a review from nickvergessen June 30, 2021 09:07
@Pytal Pytal dismissed nickvergessen’s stale review June 30, 2021 17:45

Fix has been commited :)

@Pytal Pytal requested a review from skjnldsv June 30, 2021 17:47
@skjnldsv skjnldsv merged commit 858da2e into stable22 Jul 1, 2021
@skjnldsv skjnldsv deleted the backport/27474/stable22 branch July 1, 2021 06:31
@blizzz blizzz mentioned this pull request Jul 1, 2021
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants