-
Notifications
You must be signed in to change notification settings - Fork 5
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
Surge Alert Status and Ordering based on status and opens fields #2040
Conversation
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.
lets add test case as well
5bd94a8
to
2d2d2b7
Compare
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.
Looks good. Few comments
- Mock timezone.now
c37022e
to
55455b3
Compare
Hey @szabozoltan69, could you review and merge/deploy this to staging? |
Hey @szabozoltan69 We also need to run this after deployment to initiate the status. After that, it will be updated every 12 hours. ./manage.py update_surge_alert_status |
Hey @thenav56 |
@szabozoltan69 Can you run |
Now I did it. |
Addresses
status
andopens
fields #2036Changes
Post deployment actions
@szabozoltan69 We need to run this after deployment to update the status. After that, it will be updated every 12 hours.
Checklist
Things that should succeed before merging.
Release
If there is a version update, make sure to tag the repository with the latest version.