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.

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

Record Session start time #1028

Open
nmcharlton opened this issue Oct 30, 2022 · 2 comments
Open

Record Session start time #1028

nmcharlton opened this issue Oct 30, 2022 · 2 comments

Comments

@nmcharlton
Copy link

Is your feature request related to a problem? Please describe.
We need to uniquely identify sessions in the Admin Panel in a way that's meaningful for users.
Wallet name + session start date and time seems like a good way of doing so.

At the moment we can either derive the captured_at date/time for the earliest capture in the session (cumbersome), or use the created_at session time (potentially confusing for users).

Describe the solution you'd like
Record and upload the start time of each tracking session as part of the session information.

Describe alternatives you've considered
We could just derive a pseudo-start time using the first captured_at time in the bulk pack pipeline, but I'm not clear on whether the start of the session and first capture are considered as separate events in the app.

Additional context
Greenstand/treetracker-admin-client#896 describes the need to filter captures by session.
Eventually, we will add a Tracking Sessions page in the Admin Panel that will display a summary of the session and act as a starting point for verification and analysis.

@Kpoke
Copy link

Kpoke commented Nov 12, 2022

A start_time field for sessions has been integrated into the dev environment's ingestion pipeline.

@Elforama
Copy link
Contributor

Elforama commented Dec 8, 2022

Notes for Android work:

  • Session start time is already captured and saved
  • Session start time is not part of the SessionRequest object
  • Add session time to the request as 'start_time'

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

No branches or pull requests

4 participants