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

Implement Entity Wrappers #65

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Implement Entity Wrappers #65

merged 1 commit into from
Jan 8, 2024

Conversation

rgreinho
Copy link
Member

@rgreinho rgreinho commented Jan 6, 2024

Implements entity wrappers to be used in the API instead of using the
models directly. Each wrapper also implement the IntoActiveModel trait
to handle to conversion between the Wrapper and the ActiveModel.

Signed-off-by: Rémy Greinhofer [email protected]

@rgreinho rgreinho self-assigned this Jan 6, 2024
@pull-request-size pull-request-size bot added the size/M Medium (30-99 lines of changes) label Jan 6, 2024
@rgreinho rgreinho force-pushed the api-model-wrappers branch from 322f854 to d3ea750 Compare January 6, 2024 22:53
@pull-request-size pull-request-size bot added size/L Large (100-499 lines of changes) and removed size/M Medium (30-99 lines of changes) labels Jan 6, 2024
@rgreinho rgreinho force-pushed the api-model-wrappers branch 2 times, most recently from b0ef7b7 to e19ee7e Compare January 7, 2024 00:10
Implements entity wrappers to be used in the API instead of using the
models directly. Each wrapper also implement the `IntoActiveModel` trait
to handle to conversion between the Wrapper and the ActiveModel.

Signed-off-by: Rémy Greinhofer <[email protected]>
@rgreinho rgreinho force-pushed the api-model-wrappers branch from e19ee7e to 2c4bd80 Compare January 8, 2024 01:41
@rgreinho rgreinho marked this pull request as ready for review January 8, 2024 01:44
@rgreinho rgreinho merged commit e936566 into main Jan 8, 2024
7 checks passed
@rgreinho rgreinho deleted the api-model-wrappers branch January 8, 2024 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Large (100-499 lines of changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant