Skip to content
This repository has been archived by the owner on May 22, 2023. It is now read-only.

Don't remove high level bosses #46

Closed
walfie opened this issue Sep 11, 2016 · 1 comment
Closed

Don't remove high level bosses #46

walfie opened this issue Sep 11, 2016 · 1 comment
Labels
enhancement

Comments

@walfie
Copy link
Owner

walfie commented Sep 11, 2016

Currently, bosses are removed periodically if they aren't seen in a while (currently 6 hours). Some high-level bosses are rarely seen, so it could be a long time before they're seen again.

Suggestion is to never purge bosses that are lv100+
walfie added a commit that referenced this issue Sep 11, 2016
@walfie
Don't remove high level bosses
5c57af9 
Bosses higher than a certain level won't be removed on the periodic
boss cleanup. This is because HL boss tweets happen infrequently.

Closes #46
@walfie walfie mentioned this issue Sep 11, 2016
Merged
walfie added a commit that referenced this issue Sep 11, 2016
@walfie
Don't remove high level bosses (#48)
2836e80 
Bosses higher than a certain level won't be removed on the periodic
boss cleanup. This is because HL boss tweets happen infrequently.

Closes #46
@walfie
Copy link
Owner Author

walfie commented Sep 16, 2016

Actually this was only resolved on the backend -- the frontend still has a hardcoded 6 hours.

@walfie walfie reopened this Sep 16, 2016
walfie added a commit that referenced this issue Sep 16, 2016
@walfie
Don't remove old bosses in frontend
a74a4db 
Closes #46 (again)
@walfie walfie mentioned this issue Sep 16, 2016
Merged
walfie added a commit that referenced this issue Sep 16, 2016
@walfie
Don't remove old bosses in frontend (#56)
3f6dd3c 
The attempt at keeping high level bosses in #48 was backend only, I
forgot to update the frontend code.


Closes #46 (again)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@walfie