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.

Sign up for GitHub

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

Jump to bottom

backend(api): Convert fields to classes #34

Closed
Topvennie opened this issue Mar 1, 2024 · 4 comments
Closed

backend(api): Convert fields to classes #34

Topvennie opened this issue Mar 1, 2024 · 4 comments
Milestone
Final milestone

Comments

@Topvennie
Copy link
Contributor

Some models reuse the same custom fields.
It's better to make a custom class and reuse that.
@Topvennie Topvennie added this to the Backend milestone milestone Mar 1, 2024
@Topvennie Topvennie mentioned this issue Mar 1, 2024
Merged
@EwoutV
Copy link
Contributor

EwoutV commented Mar 13, 2024

@Topvennie is this still relevant?

@Topvennie
Copy link
Contributor Author

There are a few fields still being used here and there but it isn't a big issue anymore

@Topvennie
Copy link
Contributor Author

I've moved it to the final milestone as it is a minor cleanup

@Topvennie
Copy link
Contributor Author

Moved to #225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Final milestone
Development

No branches or pull requests
2 participants
@Topvennie @EwoutV