Skip to content

Use dataclasses instead of TypedDicts for response classes#112

Merged
miaucl merged 1 commit intomainfrom
dataclasses
Jan 11, 2025
Merged

Use dataclasses instead of TypedDicts for response classes#112
miaucl merged 1 commit intomainfrom
dataclasses

Conversation

@tr4nt0r
Copy link
Collaborator

@tr4nt0r tr4nt0r commented Jan 4, 2025

Refactor response classes to utilize dataclasses. Mashumaro is used for serialization of json responses

@tr4nt0r tr4nt0r marked this pull request as draft January 4, 2025 22:53
@tr4nt0r tr4nt0r requested a review from miaucl January 4, 2025 23:30
@tr4nt0r tr4nt0r marked this pull request as ready for review January 5, 2025 00:08
miaucl
miaucl previously approved these changes Jan 5, 2025
Copy link
Owner

@miaucl miaucl left a comment

Choose a reason for hiding this comment

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

Was bringt syrupy als vorteil? So komplex sind unsere Sachen hier ja nicht 😅

@tr4nt0r
Copy link
Collaborator Author

tr4nt0r commented Jan 6, 2025

Was bringt syrupy als vorteil? So komplex sind unsere Sachen hier ja nicht 😅

Das ist pure Faulheit meinerseits 😂

Copy link
Owner

@miaucl miaucl left a comment

Choose a reason for hiding this comment

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

unit tests

Signed-off-by: tr4nt0r <4445816+tr4nt0r@users.noreply.github.com>
@miaucl miaucl enabled auto-merge (squash) January 11, 2025 19:38
@miaucl miaucl merged commit c424b0e into main Jan 11, 2025
24 checks passed
@miaucl miaucl deleted the dataclasses branch January 11, 2025 19:48
tr4nt0r added a commit that referenced this pull request Mar 16, 2025
Use dataclasses instead of TypedDicts as response classes

Signed-off-by: tr4nt0r <4445816+tr4nt0r@users.noreply.github.com>
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.

2 participants