Skip to content

Conversation

@backportbot-nextcloud
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 27, 2020

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #21123

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Looks like essential parts of the original patch are missing

@blizzz
Copy link
Member

blizzz commented May 28, 2020

Looks like essential parts of the original patch are missing

changes lost were due to code style differences

pagination is taken care of properly in the search logic in Access class

Signed-off-by: Arthur Schiwon <[email protected]>
@blizzz blizzz force-pushed the backport/21123/stable18 branch from 7aaf5fa to 5f1cfbc Compare May 28, 2020 18:13
@blizzz blizzz requested a review from ChristophWurst May 28, 2020 18:29
@MorrisJobke MorrisJobke added 3. to review Waiting for reviews and removed 2. developing Work in progress labels May 28, 2020
@MorrisJobke
Copy link
Member

I guess this has to wait for 18.0.6, right?

@blizzz
Copy link
Member

blizzz commented May 29, 2020

I guess this has to wait for 18.0.6, right?

yes, no urgency

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

🐘

@rullzer rullzer merged commit 789adb9 into stable18 Jun 6, 2020
@rullzer rullzer deleted the backport/21123/stable18 branch June 6, 2020 07:46
@rullzer rullzer mentioned this pull request Jun 8, 2020
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants