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

move deployment information from basic information block. #1214

Merged

Conversation

Alice1319
Copy link
Contributor

@Alice1319 Alice1319 commented Oct 11, 2024

fixes eclipse-xpanse/xpanse#1944

for isv:
register panel:
image

catalog:
image

catalog update:
image

for csp:
review:
image

details:
image

registered service:
image

@baixinsui
Copy link
Contributor

LGTM

1 similar comment
@dengyali0125
Copy link

LGTM

@dengyali0125
Copy link

LGTM

@Alice1319 Alice1319 force-pushed the feature/move-deployment-information branch from d793231 to 1be91fe Compare October 11, 2024 09:12
@Alice1319 Alice1319 force-pushed the feature/move-deployment-information branch from 1be91fe to 158ddb3 Compare October 12, 2024 09:54
@Alice1319
Copy link
Contributor Author

Alice1319 commented Oct 14, 2024

image

image

make deployment variables' description rows keep same height.

@Alice1319
Copy link
Contributor Author

image

boolean type value must be empty if nothing is received from backend: fixed.

@Alice1319 Alice1319 force-pushed the feature/move-deployment-information branch from 158ddb3 to bcf30fa Compare October 14, 2024 06:35
return (
<Popover
content={<pre className={oclDisplayStyles.oclDeploymentScript}>{yamlDocument.toString()}</pre>}
title={'Deployment'}
title={'Service Availability Configs'}
Copy link
Contributor

Choose a reason for hiding this comment

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

Service Availability Config

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

fixed.

@Alice1319 Alice1319 force-pushed the feature/move-deployment-information branch from bcf30fa to cca4a3c Compare October 14, 2024 07:50
Copy link
Contributor

@swaroopar swaroopar left a comment

Choose a reason for hiding this comment

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

LGTM

@swaroopar swaroopar merged commit 031f64c into eclipse-xpanse:main Oct 14, 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.

Move deployment information from basic information block
4 participants