Skip to content

bugfix(web-app-files): [OCISDEV-775] shared with does not show members#13667

Merged
mzner merged 1 commit into
masterfrom
bugfix/OCISDEV-775/shared-with-does-not-show-members
Apr 16, 2026
Merged

bugfix(web-app-files): [OCISDEV-775] shared with does not show members#13667
mzner merged 1 commit into
masterfrom
bugfix/OCISDEV-775/shared-with-does-not-show-members

Conversation

@mzner

@mzner mzner commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

We've fixed an issue where the list of external users in the "Shared with" section was not displayed.

Description

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

Open tasks:

  • ...

@mzner mzner requested a review from LukasHirt April 14, 2026 08:31
@mzner mzner force-pushed the bugfix/OCISDEV-775/shared-with-does-not-show-members branch from fec3a7c to 2e69cb5 Compare April 14, 2026 08:32

@LukasHirt LukasHirt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Code-wise it looks good but I have no idea about the actual fix here. What was the reason that the list was not displayed? Are these changes safe since they're dropping two specific guardrails?

@mzner mzner force-pushed the bugfix/OCISDEV-775/shared-with-does-not-show-members branch from 2e69cb5 to d9679f2 Compare April 16, 2026 08:19
@sonarqubecloud

Copy link
Copy Markdown

@mzner mzner merged commit 3f543b6 into master Apr 16, 2026
5 checks passed
@mzner mzner deleted the bugfix/OCISDEV-775/shared-with-does-not-show-members branch April 16, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants