Skip to content
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.

Sign up for GitHub

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

Jump to bottom

RTM: Poll periodically, not alongside Active Alerts request #68

Closed
Maelstromeous opened this issue Nov 10, 2020 · 0 comments · Fixed by #69
Closed

RTM: Poll periodically, not alongside Active Alerts request #68

Maelstromeous opened this issue Nov 10, 2020 · 0 comments · Fixed by #69
Assignees
@Maelstromeous
Labels
bug Something isn't working p2 Disruptive issues but not show stoppers
Milestone
Website 3.0 Alpha

Comments

@Maelstromeous
Copy link
Member

Atm we're excessively querying the population data every 5 seconds when it should be every 60. Need to stop the calls piggybacking off the active alerts call and implement it's own method of data refresh every 60 seconds.
@Maelstromeous Maelstromeous added bug Something isn't working p2 Disruptive issues but not show stoppers labels Nov 10, 2020
@Maelstromeous Maelstromeous added this to the Website 3.0 MVP milestone Nov 10, 2020
@Maelstromeous Maelstromeous self-assigned this Nov 10, 2020
@Maelstromeous Maelstromeous mentioned this issue Nov 10, 2020
Merged
Maelstromeous added a commit that referenced this issue Nov 10, 2020
@Maelstromeous
Merge pull request #69 from ps2alerts/fix/68-RTM-pops-polling
de01946 
#68 - RTM moved alertpops calls to be on it's own interval
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Maelstromeous Maelstromeous

Labels
bug Something isn't working p2 Disruptive issues but not show stoppers
Projects
None yet
Milestone
Website 3.0 Alpha
Development

Successfully merging a pull request may close this issue.

#68 - RTM moved alertpops calls to be on it's own interval ps2alerts/website
1 participant
@Maelstromeous