-
Notifications
You must be signed in to change notification settings - Fork 15
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
Apply Upcoming Django Security Patch #427
Comments
@farhaanbukhsh tagging you here for the review and merge of these PRs |
PR in edx-platform is also ready for review/merge, all the tests were successful |
@magajh I have reviewed it and I am going to merge it too. :) |
I can merge all of it but I am waiting for the CCs of those repos to do that! |
@farhaanbukhsh Got it! But I think that’s unlikely, especially with the release so close. For past security patches, it’s usually the release manager who tests and merges most of the PRs – for example, #386 |
@magajh No issues will merge them 👍🏾 |
Awesome, thank you! we should incorporate this patch into this release cc @cmltaWt0 |
@magajh We will :) I am testing these PRs by building the image :) |
Thanks so much, @farhaanbukhsh! I'm closing this issue now that all the PRs have been merged |
Apply upcoming Django security release
https://groups.google.com/g/django-announce/c/gmyqS48O138
Open edX services to upgrade (taken from https://openedx.atlassian.net/wiki/spaces/COMM/pages/4558782480/Sumac.master)
The text was updated successfully, but these errors were encountered: