Skip to content

removes conditional render option to edit description on wf/task metad… #1897

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

wouter1975
Copy link
Collaborator

@wouter1975 wouter1975 commented May 1, 2025

removes conditional render option to edit description on wf/task metadata page, requires orch-core 4.0.0

Note: align the release of this feature with the core release, and combine it with the breaking change of the TARGET=validation feature.

Copy link

changeset-bot bot commented May 1, 2025

🦋 Changeset detected

Latest commit: af7cf39

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@orchestrator-ui/orchestrator-ui-components Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

},
// description: {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Either uncomment this or remove it. Is there a reason to keep it?

@@ -258,7 +258,7 @@ export interface Step {
export interface WorkflowDefinition {
workflowId: string;
name: string;
description?: string;
description: string;
Copy link
Collaborator

Choose a reason for hiding this comment

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

By right, this is a breaking change. I also saw that this type is used in multiple other places:
image

Did you verify all of them? Same goes for the TaskDefinition type below

description={value}
/>
) : null,
renderData: (value, row) => (
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do not forget to add changeset to this PR

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants