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

Health Specifications page #939

Open
rstjames opened this issue Jun 18, 2024 · 11 comments
Open

Health Specifications page #939

rstjames opened this issue Jun 18, 2024 · 11 comments

Comments

@rstjames
Copy link

rstjames commented Jun 18, 2024

Please go to this doc and fill in the needed information to create each page.
https://docs.google.com/document/d/1DpBEpd8fEzaOsvr6HndwYwu1jzFSphiefZNAMxp2Gs4/edit#heading=h.4kitmexapqcp

@rstjames
Copy link
Author

@jclingan Please review and leave screenshots with comments. Some info might be missing if it was not supplied:
https://microprofile.io/specifications/health/
Pass: MPreview

@Emily-Jiang
Copy link
Member

@rstjames the maven coordinates were wrong. It shows JWT. Please get the snippet from the Health doc. Also, the page does not list the compatible products.

@rstjames
Copy link
Author

maven coordinates fixed

@Emily-Jiang
Copy link
Member

Looks good.

@xstefank
Copy link
Member

It now shows rest-client coordinates in the preview, should be:

<dependency>
    <groupId>org.eclipse.microprofile.health</groupId>
    <artifactId>microprofile-health-api</artifactId>
    <version>4.0</version>
</dependency>

@xstefank
Copy link
Member

@rstjames also probably this is also coming from REST client, we have only one mandatory TCK in Health.

2024-09-13-1726210833_261x146

@rstjames
Copy link
Author

updated

@rstjames
Copy link
Author

rstjames commented Oct 3, 2024

For review:
https://microprofile.io/specifications/health/
Password: MPreview

What I need are links to git for old versions

@Emily-Jiang @jclingan @edbratt @aeiras

@rstjames
Copy link
Author

rstjames commented Oct 9, 2024

@xstefank please review and provided the missing links for older version thanks

@rstjames
Copy link
Author

Updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants