-
Notifications
You must be signed in to change notification settings - Fork 128
WIP: Stations #1386
base: async
Are you sure you want to change the base?
WIP: Stations #1386
Conversation
Forcing last object seems to work - they seems to be in order in protos.
|
Many thx for that. I guess this also requires some work on map gui for the users, e.g. RocketMAD? |
Missing declaration out of scope
DB Migration
Was too lazy to generate file again.
|
You can test this now, shouldn't explode MAD. |
|
Seems like old maxbattle data is still in proto after 21:00 if powerspot is valid next day. If it disappears 5:00 the maxbattle data is gone. |
|
Added webhook sending data, just 1:1 fields in proto(names). I don't see any PR in PoracleJS so we will adjust it later I guess. We need to bump REDIS cache key to something more than 15 minutes - there is no need to update those every 15 minutes. |
No need to change to json and to json again
Should change nothing - stations updates when battle_end or end_time changes - it will just decrease number of webhooks for the same station without any change. However if frontend is using that field to show something like last_scanned in X hours it could break that.
|
There is a chance we will need to adjust redis cache key to include battle_pokemon.pokemon_id or bread_battle_seed. I hope we can do bread_battle_seed tbh as pokemon_id could be not enough if Niantic one day decide to run (f.e.) both Ponyta's in some raid day or something - hour each - those will only differ by form_id. Or normal Pikachu and some costume one. |
We don't need to, but I have a feeling we still should use battle_seed. |
|
Adjusted database schema so it will work with reactMap. No migration, you need to drop table and set alembic_version back to 'async'. before using new version of this PR. |
Just truncate station name at 120/100 chars - I don't think any frontend want to display 250 characters long station name, on mobile.
Not really, they expect |
|
Seems like Niantic yet again spawned phantom stations (even better - for next day) and then deleted them ASAP? |
|
and another wave of spawns of powerspots for tomorrow however not deleted today. Third time a charm? N figured out how it works? :D |
Added Stations to database (not in mapview/stats).
Fixed routes not saving due to changed proto, from my test seems like taking last element is enough - it's accepted or it's not in protos, at all.
Fixed raids not saving due to changed proto (
is_exclusiveis gone, force 0)Added ssl=False to RestHelper for supported versions query and outdated cert there.
Added
Baseto alembic env so can generate automigration in future.#TODO:
stats? /do we really need stats for this and I don't really want to touch stats, especially webview/