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

ECOPROJECT-2318: Add link to return to Assisted Migration when credentials were accepted #37

Merged

Conversation

ammont82
Copy link
Collaborator

@ammont82 ammont82 commented Nov 8, 2024

Related to https://issues.redhat.com/browse/ECOPROJECT-2318

Once the discovery vm has completed discovery of the vcenter env and reported to the migration assessment service, the discovery UI shows no indication that the discovery is complete, or what to do next.

We add a link back to the migration assessment UI when credentials were accepted:

image

Signed-off-by: Montse Ortega <mortegag@redhat.com>
@ammont82 ammont82 force-pushed the ECOPROJECT-2318-discovery-ui-vm-return-to-assisted-migration branch from accf365 to e9141a4 Compare November 8, 2024 10:55
Signed-off-by: Montse Ortega <mortegag@redhat.com>
@ammont82 ammont82 force-pushed the ECOPROJECT-2318-discovery-ui-vm-return-to-assisted-migration branch from 0eb5938 to 4c1983b Compare November 11, 2024 09:19
Signed-off-by: Montse Ortega <mortegag@redhat.com>
@ammont82 ammont82 force-pushed the ECOPROJECT-2318-discovery-ui-vm-return-to-assisted-migration branch from 29ae5ca to cc30b70 Compare November 11, 2024 13:12
Signed-off-by: Montse Ortega <mortegag@redhat.com>
@yuvalgalanti
Copy link

Console_migrate clusters_06_a
Console_migrate clusters_06_b
Console_migrate clusters_06_c

I updated the wording to fit our current microscopy, and changed the current connected state to 2 statuses gathering inventory and Discovery completed

Signed-off-by: Montse Ortega <mortegag@redhat.com>
@ammont82 ammont82 merged commit f1e6666 into main Nov 15, 2024
3 checks passed
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.

3 participants