-
Notifications
You must be signed in to change notification settings - Fork 3
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
Chapter 5: MicroProfile Configuration #14
base: main
Are you sure you want to change the base?
Conversation
Understanding MicroProfile Configuration Different environments required for Microservices development Working with Various Configuration Sources Key capabilities of MicroProfile Configuration
Adding first 4 sections to chapter05.adoc
I am reviewing this PR and will provide feedback asap. |
Awaiting review feedback and approval. |
private boolean maintenanceMode; | ||
---- | ||
|
||
In this example, the `product.maintenanceMode` property will be converted to an `Boolean` before it is injected into the `maintenanceMode` field. If the conversion fails, a `org.eclipse.microprofile.config.ConversionFailedException` will be thrown. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The exception does not exist though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was not resolved.
} | ||
---- | ||
|
||
In this example, we’re injecting a property named "product.maintenanceMessage" into the message field of our MaintenanceMessage class. If the property does not exist, a `org.eclipse.microprofile.config.ConfigPropertyNotFoundException` will be thrown. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The exception does not exist.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still see this exception on the line 325.
Co-authored-by: Emily Jiang <emijiang6@googlemail.com>
Co-authored-by: Emily Jiang <emijiang6@googlemail.com>
Minor grammar fixes
@ttelang please comment here after you have addressed all of the comments |
I have addressed all of the comments. @Emily-Jiang - Kindly merge this PR |
I double checked and found 2 comments were not resolved. Please address them. Thanks! |
Adding Chapter 5 about MicroProfile Configuration