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

Verify/Captures: Filter by session #896

Closed
nmcharlton opened this issue Oct 16, 2022 · 3 comments · Fixed by #1022
Closed

Verify/Captures: Filter by session #896

nmcharlton opened this issue Oct 16, 2022 · 3 comments · Fixed by #1022
Labels
size: medium Medium-size issue tool: Captures Relates to the capture data tool (Captures page, formerly Trees) tool: Verify Relates to the capture verification tool (Verify page) type: enhancement New feature or request UI/UX Requires some UI/UX design input

Comments

@nmcharlton
Copy link
Collaborator

New Feature

A filter in the Verify and Captures tools that allows the user to see all captures from a specific tracking session.

Screenshot 2022-10-16 at 16 45 04

Desired Behaviour

  1. In the filter, users see a list of sessions labeled with start date/time (session.created_at) and wallet name (session.wallet). E.g., 2022-10-16 16:54 | treetracker@sustainably.run.
  2. Dates are displayed using the locale format, as with other dates in the Admin Panel.
  3. Sessions are sorted by created_at with the most recent at the top.
  4. There will be thousands of sessions, so only the first 50 are displayed initially.
  5. Users can search the list by wallet name or date (similar to the Tag filter).
  6. We only show the sessions the user has permission to view, according to their organization (using session.organization_id). The implementation for this should follow the pattern for other filters (e.g., Organization), which may require some coordination with the Keycloak implementation (Keycloak auth #261, Keycloak project).

Sessions

The list of sessions can be queried from query API ($REACT_APP_FIELD_DATA_API_ROOT/session/, see field-data spec).

Dependencies

Depends on Greenstand/treetracker-query-api#244

@nmcharlton nmcharlton added type: enhancement New feature or request tool: Verify Relates to the capture verification tool (Verify page) UI/UX Requires some UI/UX design input size: medium Medium-size issue tool: Captures Relates to the capture data tool (Captures page, formerly Trees) labels Oct 16, 2022
@maxt0214
Copy link
Contributor

@nmcharlton
I have added a component for session list, but I am having cors issue with the old api $REACT_APP_FIELD_DATA_API_ROOT/session/

The error is here:

Sessions

I followed getOrganizations to getSessions:

sessionAPI

Is the CORS issue on the server side? If so, I could try to first work on Greenstand/treetracker-query-api#244

@nmcharlton
Copy link
Collaborator Author

CORS issue captured in Greenstand/treetracker-field-data#86

@nmcharlton
Copy link
Collaborator Author

@maxt0214 Please go ahead and start on Greenstand/treetracker-query-api#244 if you haven't already. Add a comment to the issue so it can be assigned to you and shout if you need any help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: medium Medium-size issue tool: Captures Relates to the capture data tool (Captures page, formerly Trees) tool: Verify Relates to the capture verification tool (Verify page) type: enhancement New feature or request UI/UX Requires some UI/UX design input
Projects
Development

Successfully merging a pull request may close this issue.

2 participants