Skip to content

fix(ui): display Bitbucket Server event source icon in event flow. Fixes #13386#13387

Merged
terrytangyuan merged 1 commit into
argoproj:mainfrom
shyoon-devops:add-bitbucketserver-ui-for-event-flow
Jul 24, 2024
Merged

fix(ui): display Bitbucket Server event source icon in event flow. Fixes #13386#13387
terrytangyuan merged 1 commit into
argoproj:mainfrom
shyoon-devops:add-bitbucketserver-ui-for-event-flow

Conversation

@shyoon-devops

Copy link
Copy Markdown
Contributor

Fixes #13386

References

References argoproj/argo-events#1927

Motivation

Add support for displaying Bitbucket Server event source icon in the Argo event flow UI.

Modifications

Added icon support for Bitbucket Server event source.
image

Verification

  • Tested the UI to ensure the Bitbucket Server event source icon is displayed correctly.
  • Built the Docker image and applied it to my k8s cluster to verify.
  • Attached screenshots for UI changes.

…on in the Argo event flow UI.

Signed-off-by: sh.yoon <sh.yoon.devops@gmail.com>
@shyoon-devops shyoon-devops changed the title fit(ui): Add support for displaying Bitbucket Server event source icon in the Argo event flow UI. feat(ui): Add support for displaying Bitbucket Server event source icon in the Argo event flow UI. Fixes #13386 Jul 23, 2024
@shyoon-devops shyoon-devops changed the title feat(ui): Add support for displaying Bitbucket Server event source icon in the Argo event flow UI. Fixes #13386 feat(ui): Add support for displaying Bitbucket Server event source icon in the Argo event flow UI. Fixes #13386 (#13386) Jul 23, 2024

@agilgur5 agilgur5 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for tracking this down and fixing this! This looks good to me, but I haven't worked with the Event Flow UI, so I'll let someone else double check. EDIT: asked for a second review on the contributors channel

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 23, 2024
@shyoon-devops

shyoon-devops commented Jul 23, 2024

Copy link
Copy Markdown
Contributor Author

Sure! Thanks for your review and lgtm :)

@terrytangyuan terrytangyuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@terrytangyuan
terrytangyuan merged commit c16c5e4 into argoproj:main Jul 24, 2024
@shyoon-devops
shyoon-devops deleted the add-bitbucketserver-ui-for-event-flow branch July 29, 2024 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ui lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

argoproj/argo-events#1927: event flow diagram not showing Bitbucket eventsource icon

3 participants