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

Add another overload to path function #234

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

3wille
Copy link

@3wille 3wille commented Apr 15, 2024

Allows to pass NinjaAPI's .urls to Django's path function.

Adding URLPattern to IncludedURLConf didn't work. My guess is that this is caused by the str | None in IncludedURLConf which is not as strict as the return type of ninja_api.urls

fixes #173

Allows to pass NinjaAPI's ``.urls`` to Django's ``path`` function.

Adding ``URLPattern`` to ``IncludedURLConf`` didn't work. My guess is
that this is caused by the `str | None` in ``IncludedURLConf`` which is
not as strict as the return type of ``ninja_api.urls``

fixes sbdchd#173
@kodiakhq kodiakhq bot merged commit 01e4eb5 into sbdchd:main Apr 15, 2024
2 checks passed
@sbdchd
Copy link
Owner

sbdchd commented Apr 15, 2024

Thank you!

@3wille 3wille deleted the path-overload-ninja branch April 16, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Django-ninjas urls have the wrong shape for django.urls.path
2 participants