Skip to content

Comments

Fix Don't show groups in share placeholder if group sharing is disabled #2242#2419

Merged
ChristophWurst merged 1 commit intonextcloud:mainfrom
Zishan-7:group-share-enable-placeholder
May 22, 2023
Merged

Fix Don't show groups in share placeholder if group sharing is disabled #2242#2419
ChristophWurst merged 1 commit intonextcloud:mainfrom
Zishan-7:group-share-enable-placeholder

Conversation

@Zishan-7
Copy link
Contributor

Fix #2242 by checking group sharing enabled or not then updating the value of placeholder based on that value

@Zishan-7 Zishan-7 force-pushed the group-share-enable-placeholder branch from c127cd3 to b396367 Compare August 30, 2021 13:03
@codecov
Copy link

codecov bot commented Aug 30, 2021

Codecov Report

Patch coverage has no change and project coverage change: -1.83 ⚠️

Comparison is base (73ef6e2) 1.82% compared to head (02a003b) 0.00%.

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

Additional details and impacted files
@@            Coverage Diff             @@
##              main   #2419      +/-   ##
==========================================
- Coverage     1.82%   0.00%   -1.83%     
  Complexity     272     272              
==========================================
  Files          112      25      -87     
  Lines         6091     826    -5265     
  Branches      1450       0    -1450     
==========================================
- Hits           111       0     -111     
+ Misses        5865     826    -5039     
+ Partials       115       0     -115     
Impacted Files Coverage Δ
lib/Controller/PageController.php 0.00% <0.00%> (ø)

... and 87 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Zishan-7 Zishan-7 force-pushed the group-share-enable-placeholder branch from 87e8fd9 to 0810531 Compare August 31, 2021 07:11
@Zishan-7 Zishan-7 requested a review from skjnldsv August 31, 2021 07:13
Copy link
Member

@tcitworld tcitworld left a comment

Choose a reason for hiding this comment

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

LGTM. Can you squash your commits?

@szaimen szaimen added the 3. to review Waiting for reviews label Sep 2, 2021
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 good!

@ChristophWurst ChristophWurst force-pushed the group-share-enable-placeholder branch from 0810531 to 092dc56 Compare December 15, 2022 15:04
@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 Dec 15, 2022
@hamza221 hamza221 force-pushed the group-share-enable-placeholder branch from 092dc56 to fb0709f Compare May 22, 2023 14:45
@hamza221
Copy link
Contributor

Fixed lint check, and rebased.

@ChristophWurst ChristophWurst merged commit 1af5e02 into nextcloud:main May 22, 2023
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't show groups in share placeholder if group sharing is disabled

6 participants