Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(transfer): change TransferOption label-proptype to node #997

Merged
merged 1 commit into from
Mar 1, 2022

Conversation

Birkbjo
Copy link
Member

@Birkbjo Birkbjo commented Feb 28, 2022

Simple fix to allow more customisable Transfer-options. This is used in Data Integrations in the Scheduler app (see dhis2/scheduler-app#348). Label is rendered directly, so a PropType of node, should just work.

@Birkbjo Birkbjo requested review from Mohammer5 and varl February 28, 2022 17:30
@dhis2-bot
Copy link
Contributor

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

@dhis2-bot dhis2-bot temporarily deployed to netlify February 28, 2022 17:34 Inactive
@cypress
Copy link

cypress bot commented Feb 28, 2022



Test summary

558 0 0 0


Run details

Project ui
Status Passed
Commit 85a5330
Started Feb 28, 2022 5:38 PM
Ended Feb 28, 2022 5:46 PM
Duration 08:30 💡
OS Linux Ubuntu - 20.04
Browser Electron 94

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@varl varl merged commit c34b33f into master Mar 1, 2022
@varl varl deleted the fix/transfer-option-label-proptype-node branch March 1, 2022 08:07
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 8.0.2 🎉

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