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

Misc feature / fixes #52

Merged
merged 3 commits into from
Oct 11, 2023
Merged

Misc feature / fixes #52

merged 3 commits into from
Oct 11, 2023

Conversation

biolds
Copy link
Contributor

@biolds biolds commented Oct 5, 2023

Hi,

This PR contains some fixes and features that I needed.

Thanks for this great project!

The error could occur when a serializer is defined as an abstract class for other
serializers.
This commit changes how serializer are found:
- a flag --app_name is used to lookup by application name
- all modules are searched when using --all or --app_name (instead of only
  the <app_name>.serializers module)
- -s is used to lookup serializer classes only
- this fixes handling serializers in subdirectories
@adenh93
Copy link
Owner

adenh93 commented Oct 11, 2023

@biolds Looks good, sorry for not getting to this sooner!

@adenh93 adenh93 merged commit ab3656e into adenh93:master Oct 11, 2023
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