Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Jul 7, 2022

Fixes #1105

@juliusknorr juliusknorr changed the base branch from master to stable24 July 7, 2022 15:35
@CarlSchwan
Copy link
Member

/backport to stable23

@CarlSchwan
Copy link
Member

/backport to stable22

@codecov-commenter
Copy link

Codecov Report

Merging #1106 (bcf4f5f) into stable24 (06887a2) will not change coverage.
The diff coverage is 0.00%.

❗ Current head bcf4f5f differs from pull request most recent head d289957. Consider uploading reports for the commit d289957 to get more accurate results

@@            Coverage Diff             @@
##             stable24   #1106   +/-   ##
==========================================
  Coverage        0.69%   0.69%           
  Complexity       5748    5748           
==========================================
  Files             291     291           
  Lines           19881   19881           
==========================================
  Hits              139     139           
  Misses          19742   19742           
Impacted Files Coverage Δ
lib/Model/ShareWrapper.php 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06887a2...d289957. Read the comment docs.

@skjnldsv skjnldsv merged commit 299027b into stable24 Jul 7, 2022
@skjnldsv skjnldsv deleted the bugfix/noid/no-ocp-server branch July 7, 2022 16:19
@backportbot-nextcloud
Copy link

The backport to stable22 failed. Please do this backport manually.

@skjnldsv
Copy link
Member

skjnldsv commented Jul 7, 2022

Ah, not needed for 22

$l10n = \OC::$server->get(IFactory::class)->get('circles');

@CarlSchwan
Copy link
Member

Ah, not needed for 22

$l10n = \OC::$server->get(IFactory::class)->get('circles');

Actually just did in a separate pull #1107 as there was a small backport conflict

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.

5 participants