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

Changes to ISV view in UI to show current status of the service template #1345

Conversation

WangLiNaruto
Copy link
Contributor

@WangLiNaruto WangLiNaruto commented Dec 5, 2024

Fixed eclipse-xpanse/xpanse#2110

Register success:
image
image

Audit:
image
image

Delete:
image

Unregister:
image
image

Reregister:
image
image

Update:
image
image

@Alice1319
Copy link
Contributor

LGTM

@WangLiNaruto WangLiNaruto force-pushed the feature/current_service_status branch from 911402f to 891b743 Compare December 9, 2024 01:44
@WangLiNaruto WangLiNaruto force-pushed the feature/current_service_status branch 2 times, most recently from d249e7f to ce97006 Compare December 9, 2024 11:32
@swaroopar swaroopar self-requested a review December 9, 2024 13:31
@WangLiNaruto WangLiNaruto force-pushed the feature/current_service_status branch from 2875af7 to c177518 Compare December 10, 2024 11:40
@WangLiNaruto WangLiNaruto force-pushed the feature/current_service_status branch from c177518 to 1155580 Compare December 10, 2024 11:42
{(activeServiceDetail.serviceTemplateRegistrationState === 'in-review' ||
activeServiceDetail.serviceTemplateRegistrationState === 'approved') &&
!activeServiceDetail.availableInCatalog ? (
<Badge.Ribbon
Copy link
Contributor

Choose a reason for hiding this comment

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

still the display is wrong when update review is in progress and the original template is still available in catalog. I see only 'review in progress'.

And more over, re-register button is enabled in this case. Which is also wrong.

image

@WangLiNaruto WangLiNaruto force-pushed the feature/current_service_status branch from d76011e to 1155580 Compare December 12, 2024 02:43
@WangLiNaruto WangLiNaruto deleted the feature/current_service_status branch December 16, 2024 03:27
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.

Changes to ISV view in UI to show current status of the service template
4 participants