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

feat(Templates): Blank workflow card content change on existing workflow #6507

Merged
merged 5 commits into from
Jan 30, 2025

Conversation

Elaina-Lee
Copy link
Contributor

@Elaina-Lee Elaina-Lee commented Jan 29, 2025

Type of Change

  • Bug fix
  • Feature
  • Other

Updating the wordings of blank workflow card with contents reviewed by @ecfan
The workflow card will have a wording to let users know it will be replacing the current workflow in case of consumption where workflow already exists.

Test Plan

To ensure backward compatibility, if no parameter is provided, the blank workflow card content defaults to "start with empty workflow...". Only when it is the case of consumption and the workflow data is present, it will change the wording in the card. Added in the blank workflow card tests to check for the default message present.

Screenshots or Videos (if applicable)

On regular cases:
image

On overwriting the existing workflow case:
image

@Elaina-Lee Elaina-Lee marked this pull request as ready for review January 29, 2025 22:45
@Elaina-Lee Elaina-Lee requested a review from hartra344 as a code owner January 29, 2025 22:45
@Elaina-Lee Elaina-Lee merged commit 480b0f2 into main Jan 30, 2025
12 checks passed
@Elaina-Lee Elaina-Lee deleted the elaina/blankWorkflowReText branch January 30, 2025 00:28
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.

4 participants