pdpinch commented 1 year ago

Pre-Flight checklist

[ ] Testing
- [ ] Code is tested
- [ ] Changes have been manually tested

What are the relevant tickets?

part of #5212

What's this PR do?

Stops updating user dashboard data every day. Only updates user dashboard data on Fridays.

How should this be manually tested?

I don't think it can be manually tested

Any background context you want to provide?

We need to authorize exams on Tuesday 8/16. We don't want the exam authorization task to be slowed down waiting for these dashboard updates.

codecov-commenter commented 1 year ago

Codecov Report

Merging #5217 (9a084c4) into master (4ad58f7) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 9a084c4 differs from pull request most recent head 4502d8e. Consider uploading reports for the commit 4502d8e to get more accurate results

@@           Coverage Diff           @@
##           master    #5217   +/-   ##
=======================================
  Coverage   93.98%   93.98%           
=======================================
  Files         499      499           
  Lines       23163    23163           
  Branches     1415     1415           
=======================================
  Hits        21769    21769           
  Misses       1289     1289           
  Partials      105      105

Impacted Files	Coverage Δ
micromasters/settings.py	`89.75% <ø> (ø)`

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

pdpinch commented 1 year ago

I accepted your change. If we change the task name, does it need to be updated anywhere else? I searched but didn't find anything.

arslanashraf7 commented 1 year ago