Skip to content

Conversation

@flaminic
Copy link
Contributor

@flaminic flaminic commented Dec 11, 2025

Description

With this PR we want to change the sharing dialog so that it can be used for only data sharing, and no metadata sharing.
This will be used in the new maintanace app, for program stages sharing setting.

Checklist

  • API docs are generated
  • Tests were added
  • Storybook demos were added

Screenshots

Screenshot 2025-12-11 at 14 20 15

@flaminic flaminic requested a review from a team as a code owner December 11, 2025 12:37
@flaminic flaminic marked this pull request as draft December 11, 2025 12:37
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Dec 11, 2025

🚀 Deployed on https://pr-1709--dhis2-ui.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify December 11, 2025 12:41 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify December 11, 2025 13:30 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify December 12, 2025 16:35 Inactive
@henrikmv henrikmv marked this pull request as ready for review December 13, 2025 07:20
@dhis2-bot dhis2-bot temporarily deployed to netlify December 22, 2025 09:00 Inactive
dataSharing
? 'endWrapper endWrapperData'
: 'endWrapper endWrapperMetadata'
dataSharing && metadataSharing
Copy link
Collaborator

Choose a reason for hiding this comment

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

minor(if it makes sense to you): maybe all of these dataSharing && metadataSharing combinations can be stored in a variable to make it less verbose and semantic .. something like showAllSharingModes or even showDataAndMetadataSharing (or the other way, showMetaDataOnly)

@sonarqubecloud
Copy link

@flaminic flaminic merged commit 56dc592 into master Dec 22, 2025
19 checks passed
@flaminic flaminic deleted the sharing-dialog-to-hide-metadata-sharing branch December 22, 2025 14:09
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 10.11.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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