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): removing unneeded ':' in protocol for workflow-event-bindings #4893

Merged
merged 1 commit into from
Jan 15, 2021

Conversation

igaskin
Copy link
Member

@igaskin igaskin commented Jan 15, 2021

Super minor typo fix. The URL will still be valid in a browser with ::, but still good to correct.
Screen Shot 2021-01-15 at 2 33 47 PM

Signed-off-by: Isaac Gaskin <igaskin@tesla.com>
@igaskin igaskin force-pushed the user/igaskin/document_protocol_typo branch from df0c896 to a23e405 Compare January 15, 2021 22:38
@alexec alexec changed the title fix(workflow-event-bindings): removing unneeded ':' in protocol fix(ui): removing unneeded ':' in protocol for workflow-event-bindings Jan 15, 2021
@alexec
Copy link
Contributor

alexec commented Jan 15, 2021

LGTM

@alexec alexec merged commit 1f13241 into argoproj:master Jan 15, 2021
@igaskin igaskin deleted the user/igaskin/document_protocol_typo branch January 15, 2021 23:28
This was referenced Jan 19, 2021
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.

2 participants