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(ui): display Bitbucket Server event source icon in event flow. Fixes #13386 #13387

Conversation

shyoon-devops
Copy link
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.

@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 changed the title feat(ui): Add support for displaying Bitbucket Server event source icon in the Argo event flow UI. Fixes #13386 (#13386) feat(ui): display Bitbucket Server event source icon in event flow. Fixes #13386 (#13386) Jul 23, 2024
@agilgur5 agilgur5 changed the title feat(ui): display Bitbucket Server event source icon in event flow. Fixes #13386 (#13386) feat(ui): display Bitbucket Server event source icon in event flow. Fixes #13386 Jul 23, 2024
Copy link
Member

@agilgur5 agilgur5 left a comment

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
@agilgur5 agilgur5 changed the title feat(ui): display Bitbucket Server event source icon in event flow. Fixes #13386 fix(ui): display Bitbucket Server event source icon in event flow. Fixes #13386 Jul 23, 2024
@shyoon-devops
Copy link
Contributor Author

shyoon-devops commented Jul 23, 2024

Sure! Thanks for your review and lgtm :)

Copy link
Member

@terrytangyuan terrytangyuan left a comment

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
20 of 21 checks passed
@shyoon-devops shyoon-devops deleted the add-bitbucketserver-ui-for-event-flow branch July 29, 2024 07:10
agilgur5 pushed a commit that referenced this pull request Jul 30, 2024
@agilgur5 agilgur5 added this to the v3.5.x patches milestone Jul 30, 2024
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