Skip to content

Conversation

@timgraham
Copy link
Collaborator

No description provided.

@aclark4life
Copy link
Collaborator

aclark4life commented Oct 16, 2025

I'll add these to the project template, I wonder why we didn't notice sessions which has to be installed for Django to run but currently doesn't have a custom apps.py? I assume it's because it has a model but the model is not a critical part of the app's functionality, but I haven't looked yet.

@timgraham
Copy link
Collaborator Author

The session model uses session_key = models.CharField(_("session key"), max_length=40, primary_key=True) rather than AutoField, so I was wrong to list it here. The other apps aren't enabled in the default project template's INSTALLED_APPS. I suppose they could be added to the template, even if unused.

@aclark4life aclark4life self-requested a review October 24, 2025 15:36
Copy link
Collaborator

@aclark4life aclark4life left a comment

Choose a reason for hiding this comment

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

LGTM

@timgraham timgraham merged commit 84dc005 into mongodb:main Oct 24, 2025
14 checks passed
@timgraham timgraham deleted the contrib-migrations branch October 24, 2025 17:47
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