Skip to content

Clearing the cache if the site's configuration was removed. - #15632

Merged
afzaledx merged 1 commit into
masterfrom
afzaledx/clear_cached_programs
Jul 21, 2017
Merged

Clearing the cache if the site's configuration was removed.#15632
afzaledx merged 1 commit into
masterfrom
afzaledx/clear_cached_programs

Conversation

@afzaledx

Copy link
Copy Markdown

If the site's configuration does not contain the API URL, it should not keep on displaying the old cached list of uuids.
LEARNER-1146

The site's configuration does not contain the API URL, it should not keep on displayed the old cached list of uuids.
LEARNER-1146
@afzaledx
afzaledx requested a review from rlucioni July 21, 2017 13:26
@rlucioni

Copy link
Copy Markdown
Contributor

@afzaledx please add a test verifying this behavior.

@afzaledx

Copy link
Copy Markdown
Author

@rlucioni , I will add this to the cleanup PR https://github.com/edx/edx-platform/pull/15451 which removes the waffle switches.

@afzaledx
afzaledx merged commit 17d5386 into master Jul 21, 2017
@rlucioni
rlucioni deleted the afzaledx/clear_cached_programs branch July 21, 2017 16:43
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production on Monday, July 24, 2017.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been rolled back from the production environment.

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.

3 participants