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

Replace WorkloadsOverviewDetailRegistry with reactive solution #4817

Merged
merged 1 commit into from
Mar 4, 2022

Conversation

jansav
Copy link
Contributor

@jansav jansav commented Feb 4, 2022

Signed-off-by: Janne Savolainen [email protected]

Closes #4816

@jansav jansav added the chore label Feb 4, 2022
@jansav jansav requested a review from a team as a code owner February 4, 2022 08:38
@jansav jansav requested review from aleksfront and jim-docker and removed request for a team February 4, 2022 08:38
@jansav jansav force-pushed the workloads-overview-detail-registry branch from 349d86f to 627f37a Compare February 4, 2022 09:39
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@jansav jansav force-pushed the workloads-overview-detail-registry branch from 627f37a to 9580a1e Compare February 28, 2022 05:34
@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

Iku-turso
Iku-turso previously approved these changes Mar 3, 2022
aleksfront
aleksfront previously approved these changes Mar 3, 2022
Copy link
Contributor

@aleksfront aleksfront left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2022

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@jansav jansav dismissed stale reviews from aleksfront and Iku-turso via eb5475b March 3, 2022 14:11
@jansav jansav force-pushed the workloads-overview-detail-registry branch from 9580a1e to eb5475b Compare March 3, 2022 14:11
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2022

Conflicts have been resolved. A maintainer will review the pull request shortly.

@jansav jansav merged commit 125a47a into master Mar 4, 2022
@jansav jansav deleted the workloads-overview-detail-registry branch March 4, 2022 08:04
@Nokel81 Nokel81 mentioned this pull request May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WorkloadsOverviewDetailRegistry
4 participants