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

Update workflow status badge to use grpc enum #568

Conversation

Assem-Hafez
Copy link
Contributor

  • changing enum for workflow status tag to use the values from GRPC response (this is a temp state until we work on data transformation layer)
  • added "use client" to the status tag to be able to use it in server components.

Copy link
Contributor

@adhityamamallan adhityamamallan left a comment

Choose a reason for hiding this comment

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

LGTM, but we'll need to take a look at the linter configuration in the future.

@adhityamamallan adhityamamallan merged commit 4ca7aa9 into uber:release/4.0.0 May 16, 2024
4 checks passed
Assem-Uber pushed a commit to Assem-Hafez/cadence-web that referenced this pull request May 16, 2024
* Update workflow status badge to use grpc enum

* update unit tests to use the type too
Assem-Uber pushed a commit that referenced this pull request May 22, 2024
* Update workflow status badge to use grpc enum

* update unit tests to use the type too

* workflow page header

* lint fix

* add cluster to the url navigation

* remove formate close-event-result

* fix mock linting issue

* workflow page header

* Update workflow status badge to use grpc enum (#568)

* Update workflow status badge to use grpc enum

* update unit tests to use the type too

* remove unused format closed event file

* fix typo

* move svg after components

* fixed the mock for tabs config
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.

None yet

2 participants