-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feat: Remove DEPR waffle switch: ENABLE_GLOBAL_STAFF_OPTIMIZATION #36069
base: master
Are you sure you want to change the base?
feat: Remove DEPR waffle switch: ENABLE_GLOBAL_STAFF_OPTIMIZATION #36069
Conversation
Thanks for the pull request, @marslanabdulrauf! This repository is currently maintained by @openedx/wg-maintenance-edx-platform. Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@@ -139,9 +138,6 @@ | |||
'get_course_and_check_access'] | |||
|
|||
WAFFLE_NAMESPACE = 'studio_home' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unused. Can be removed.
045ac34
to
40e905b
Compare
87cc390
to
aa44fe2
Compare
Couple of small things:
|
e484092
to
c9eaa21
Compare
After this merge tests are failing as forked repositories don't have access to original repo secrets This will be affecting all Cc @pdpinch |
I'll take a look at the CI issue in a couple of hours. |
This should be fixed now. Re-running your unit tests... |
7fab1bf
to
8e52b3a
Compare
Thanks so much, @kdmccormick, I truly appreciate your help and swift action. |
|
I am tempted to try out this new button I have "enable auto-merge" with the expectation that it won't actually merge until the branch is rebased. But let's not take that chance. @marslanabdulrauf would mind rebasing this again and I'll plan to merge it between 8am and 9am EST tomorrow. |
8e52b3a
to
343d521
Compare
Description
This PR removes the DEPR waffle switch
ENABLE_GLOBAL_STAFF_OPTIMIZATION
and its usage.Supporting information
https://github.com/mitodl/hq/issues/6346
#35649