feat: remove waffle forced ccached miss switch

openedx / edx-django-utils

edX utilities for Django Application development.

https://edx.readthedocs.io/projects/edx-django-utils/en/latest/

Apache License 2.0

26 stars 20 forks source link

feat: remove waffle forced ccached miss switch #363

Closed mumarkhan999 closed 11 months ago

mumarkhan999 commented 11 months ago

Related Issue

https://github.com/openedx/edx-django-utils/issues/333

Description

In this PR we're removing edx_django_utils.cache.disable_forced_cache_miss_for_none waffle switch which was added in version 5.7.0 version for backward compatibility.
For more details you can have a look at changelog of version 5.7.0

robrap commented 11 months ago

@mumarkhan999: This toggle duplicated a legacy bug, right? Does that mean the bug no longer exists? It would be great if you could leave a quick note about why we no longer need the toggle. Thanks.

mumarkhan999 commented 11 months ago

@robrap Yes. The toggle is already active on PROD means None is being treated as valid cache value. More details are mentioned on the issue related to this PR.

robrap commented 11 months ago

@mumarkhan999: Great. Also, I missed the issue link if it was there already. Thanks.