Skip to content

Fix sequence creation timestamps to rely on recorded_at #675

Description

@fe51

Spotted reviewing frontend issue pyronear/new-pyro-platform#271, we have missed something in #617, mentionned #510

  • Updates how detections are linked to sequences : Currently based on the created_at field, must be related to the recorded_at field

  • check sequences fromdate endpoint to be sure it relies on started_at or if relies to created_at, updates to rely on recorded_at

In other words, the creation date of a sequence is always based on the “created at” time rather than the “recorded at” time

  • Also, related migration needs to be performed

As a result, today we can correctly display a sequence’s detections using the “recorded at” time—for example, between 7:23 p.m. and 7:26 p.m.—but the sequence’s creation date (i.e., the alert start date) is shown as 7:25 p.m. This is confusing for users.

I’m available to discuss this, but at this stage, it’s preventing us from deploying “recorded_at” on the front end. @Acruve15 @MateoLostanlen ;)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions