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

Ability to view all uploaded user records #266

Closed
kazlauskis opened this issue Oct 16, 2023 · 3 comments
Closed

Ability to view all uploaded user records #266

kazlauskis opened this issue Oct 16, 2023 · 3 comments
Milestone

Comments

@kazlauskis
Copy link
Member

Many users are unaware that the app only allows one-way uploads. As a result, when they switch devices or reinstall the app, their uploaded records list disappears. Although we added a map report to the app in the previous version, which displays all recent records on a map, it is not the same as having access to all previous records in the uploaded list, including the ability to view record details.

We've added this functionality to the ButterflyCount app, and it would be fairly simple to include it here, too.

@kitenetter kitenetter added this to the NCEA app work milestone Jan 9, 2025
@kitenetter
Copy link
Collaborator

This can be done via links to maps/data stored online

@kazlauskis
Copy link
Member Author

@kitenetter @johnvanbreda To implement this properly, we should change the warehouse API. The changes would make it backwards incompatible, so this would probably require API versioning or something similar. Here I propose a few changes:

  1. When doing a POST to occurrence/sample/media it should return a value X, doing a GET request should return a value Y, where X=Y. Unify REST API GET and POST/PUT values Indicia-Team/warehouse#520
  2. When we're doing a POST request, we most often include all sub-models and their values. This is much more efficient than uploading top samples, then each sub-sample individually, then occurrences etc. In the same way, there should be a parameter that would allow a full model request when doing a GET request. This way, apps can fetch a sample with all its nested models in a single request.
  3. Upon sample upload, sub-samples should auto-inherit all the required attributes from the parent sample if missing. We currently drop the sub-sample layer if, say, the location is missing, but this makes it much harder to deal with fetched models when the overall model structure isn't fixed. Sub-samples should auto-inherit parent sample location if missing Indicia-Team/warehouse#533
  4. Simplify model attributes, like boolean t/f to true/false, automatically derive geom and entered_sref_system from entered_sref Updating entered_sref with Sample's REST doesn't update geom Indicia-Team/warehouse#524, Make REST API return and accept model boolean attributes as true/false instead of 't'/'f' Indicia-Team/warehouse#525
  5. Warehouse GET accepts filter parameters, but only filtering on equality is supported. Allow filtering by range, like updated_on gte "2023-06-28T14:32:57+01:00".

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

No branches or pull requests

2 participants