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

Add v2 lure #65

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add v2 lure #65

wants to merge 3 commits into from

Conversation

Mygod
Copy link
Contributor

@Mygod Mygod commented Oct 1, 2021

  • Support remembering lure first seen, so that estimated lure duration can be more robust, and moved to front end.
  • Support scanning lure true expiration time when the stop is clicked.

New config options: dataparser.lure.v2 enables the new behavior and dataparser.lure.v2Webhook enables the new behavior for the webhook as well.

Changes in behavior for v2: lure_expiration_time now stores NULL if the lure is active and has unknown expiration time, and 0 if there is not a lure when the stop is last scanned.

* Support remembering lure first seen, so that estimated lure duration can be more robust, and moved to front end.
* Support scanning lure true expiration time when the stop is clicked.
@Mygod Mygod added the enhancement New feature or request label Oct 1, 2021
@Mygod Mygod requested a review from TurtIeSocks October 1, 2021 21:40
Copy link
Collaborator

@TurtIeSocks TurtIeSocks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed on Discord, seeing some issues with lure_expire_timestamp being set to the current timestamp at scan time.

@versx
Copy link
Contributor

versx commented May 7, 2022

As discussed on Discord, seeing some issues with lure_expire_timestamp being set to the current timestamp at scan time.

Should be fixed now? Lmk

@Mygod
Copy link
Contributor Author

Mygod commented May 8, 2022

Yes but @TurtIeSocks is unwilling to test it since he does not want to pull stop data from two different backends (for ReactMap).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants