fix the cache error which also occurred error on run-test #2392
Conversation
🔍 Existing Issues For ReviewYour pull request is modifying functions with the following pre-existing issues: 📄 File: website/views.py
Did you find this useful? React with a 👍 or 👎 |
|
@DonnieBLT , @arkid15r ,, there is a cache related issue I have notice , for that reason the home page not work perfectly . and also showing "run test" error in every PR . now It's fixed . can you please review for merge this update |
|
|
||
| @receiver(user_logged_out) | ||
| def handle_user_logged_out(request, user, **kwargs): | ||
| cache.clear() |
There was a problem hiding this comment.
Why do you clear the entire cache upon this event?
There was a problem hiding this comment.
yes sir, should i just remove the cache_key?
There was a problem hiding this comment.
I'm not sure what issue you fixed with that but it seem an overkill to me in any case as the entire cache is being invalidated upon any user signin/signup/logout event, right?
There was a problem hiding this comment.
yeah right sir , I will fix this up for sure
the change will fix the cache error from home page and also fix the run-test error too