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

5.0: Update django.contrib.contenttypes.prefetch #2115

Merged
merged 6 commits into from
May 8, 2024

Conversation

JaeHyuckSa
Copy link
Contributor

I have made things!

Update stubs for django.contrib.contenttypes.prefetch for Django 5.0.

  • django.contrib.contenttypes.prefetch
    • django.contrib.contenttypes.prefetch.GenericPrefetch was added

Related issues

Refs

Upstream PR

@JaeHyuckSa JaeHyuckSa changed the title 5.0: Update django.contrib.contenttypes.prefetch 5.0: Update django.contrib.contenttypes.prefetch May 3, 2024
from django.db.models.query import QuerySet

class GenericPrefetch(Prefetch):
def __init__(self, lookup: str, querysets: list[QuerySet] | None = ..., to_attr: str | None = ...) -> None: ...
Copy link
Member

Choose a reason for hiding this comment

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

Please, see django/django#18125

Copy link
Contributor Author

@JaeHyuckSa JaeHyuckSa May 3, 2024

Choose a reason for hiding this comment

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

@sobolevn I've read the corresponding PR. It's a shame that if I had thought about it a little more, I would have found a bug. Still, thank you for letting me know the good opportunity :) I'm learning again today. If Django PR merges, I'll fix the code and apply it.

@sudosubin sudosubin mentioned this pull request May 3, 2024
39 tasks
@JaeHyuckSa
Copy link
Contributor Author

I have confirmed that the code has been applied! I will fix it and commit it.

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks!

@sobolevn sobolevn merged commit c2160be into typeddjango:master May 8, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants