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.

Sign up for GitHub

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

Jump to bottom

feat!: upgrade to django 4.2 #122

Merged
merged 5 commits into from
Nov 1, 2023
Merged

feat!: upgrade to django 4.2 #122

merged 5 commits into from
Nov 1, 2023

Conversation

UsamaSadiq
Copy link
Member

@UsamaSadiq UsamaSadiq commented Sep 28, 2023 •
edited
Loading

Description

TODO

  • Verify Django is upgraded to 4.2 at all places
  • USE_DEPRECATED_PYTZ settings is added to base/common.py with an explanatory comment
  • memcached backend is changed from MemcachedCache to PyMemcacheCache in edx-internal (remote config/ argocd configs)
  • CSRF_TRUSTED_ORIGINS are updated

@grmartin grmartin force-pushed the upgrade-to-django42 branch from 98cd694 to baa064c Compare October 27, 2023 18:46
@pshiu pshiu merged commit 915f182 into main Nov 1, 2023
3 checks passed
@pshiu pshiu deleted the upgrade-to-django42 branch November 1, 2023 12:38
@pshiu
Copy link
Contributor

pshiu commented Nov 1, 2023

Thanks for this, @UsamaSadiq!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@awais786 awais786 awais786 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@UsamaSadiq @pshiu @awais786