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: 🎸 [JIRA:HCPSDKFIORIUIKIT-2716]Step Progress Indicator #787

Merged
merged 11 commits into from
Sep 13, 2024

Conversation

KevinZK
Copy link
Collaborator

@KevinZK KevinZK commented Sep 2, 2024

*Add stepProgressIndicatorNodeType for step node.

Description

Add StepProgressIndicatorNodeType for stepProgressIndicator to set the nodetype style.

+ mixture: Allow nodes to display numeric step and icon step in a mixture.
+ icon: Only allow the display of icon step.
+ text:Only allow the display of numeric step.

add TextorIcon replace Image

Add stepProgressIndicatorNodeType for step node.
@KevinZK KevinZK requested a review from a team as a code owner September 2, 2024 03:14
@KevinZK KevinZK requested review from billzhou0223 and removed request for a team September 2, 2024 03:14
@dyongxu dyongxu requested a review from xiaoyu0722 September 3, 2024 18:09
@dyongxu
Copy link
Contributor

dyongxu commented Sep 3, 2024

@xiaoyu0722 Can you review this enhancement as well ?

@KevinZK KevinZK requested a review from billzhou0223 September 5, 2024 06:29
Remove the mixture property of StepProgressIndicatorNodeType.
@KevinZK
Copy link
Collaborator Author

KevinZK commented Sep 11, 2024

CI Error:

The actions/download-artifact in CI.yml is version v1.
Is it updated to actions/download-artifact

@billzhou0223
Copy link
Contributor

CI Error:

The actions/download-artifact in CI.yml is version v1. Is it updated to actions/download-artifact

Version was taken care of by #799

@xiaoyu0722 xiaoyu0722 merged commit 9cee816 into SAP:main Sep 13, 2024
12 checks passed
@KevinZK KevinZK deleted the JIRA-2716_StepProgress branch October 21, 2024 06:40
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