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

Link documentation to the Immutables library's guide for generating immutable DynamoDB Enhanced client classes #5006

Conversation

swar8080
Copy link
Contributor

@swar8080 swar8080 commented Mar 8, 2024

Hello, the Immutables java library's website has a guide (that I wrote) on generating DynamoDB Enhanced Client classes using the immutables library. It also has a recipe for doing partial updates and projections with immutable classes.

The README has a lombok example, so maybe others would find this useful!

…ng immutable DynamoDB Enhanced Client classes
@swar8080 swar8080 requested a review from a team as a code owner March 8, 2024 02:49
@swar8080 swar8080 changed the title Link documentation to the Immutable java library's guide for generating immutable DynamoDB Enhanced client classes Link documentation to the Immutables library's guide for generating immutable DynamoDB Enhanced client classes Mar 8, 2024
@debora-ito debora-ito added the needs-review This issue or PR needs review from the team. label Apr 23, 2024
@debora-ito
Copy link
Member

@swar8080 I didn't know about this Immutables guide, good job on the DynamoDB integration page.

We usually don't reference third-party pages in our docs, because (1) the referenced page can get out-of-date or have incorrect content, and we won't have the control to change; (2) we can start getting requests to add other 3p libs.

We appreciate the work you've put into the guide, though! It helps in spreading the knowledge about DynamoDB Enhanced Client.

@debora-ito debora-ito closed this May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review This issue or PR needs review from the team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants