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

Django 4.X compatiblity #5

Open
fbinz opened this issue Sep 28, 2022 · 1 comment
Open

Django 4.X compatiblity #5

fbinz opened this issue Sep 28, 2022 · 1 comment

Comments

@fbinz
Copy link

fbinz commented Sep 28, 2022

I just found out about this cool package and wanted to test it with a current django 4.X project.
Unfortunately, it did not work out of the box, because the Shouty(AppConfig) class is not found, when registering the package only via its name shoutyorm (see the AppConfig.create method in django/apps/config.py where it specifically looks for an AppConfig in a submodule called apps - which shoutyorm does not have).

As is described in the documentation, we can be more explicit and use the app config directly in installed apps:

INSTALLED_APPS = [
    ...
    "shoutyorm.Shout",
    ...
]

This is not really an issue, just a heads up for others who might otherwise think that this package was incompatible with current django releases.

Either way: Great package and thanks for releasing it. ❤️

@kezabelle
Copy link
Owner

Hey, thanks so much for popping by and trying it out. Sorry it didn't work out of the box!

This package has always been somewhat behind, compared to the shouty-templates one because it's a bunch more fragile, so coming back to it and keeping it up to date is ... daunting.

I do have intentions to update it at some point, with some functionality rewritten to better track certain scenarios. It's all currently sitting around in a half-baked local branch which I need to get back to, and that should end up the basis for something which is full compatible with 4.x onwards -- mostly because I need 3.2+ to be well supported for work :)

Anyway. I truly appreciate you coming by and surfacing the issue.

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

No branches or pull requests

2 participants