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

uuid of capture is missing in the "Captures" Module download #928

Open
sebastiangaertner opened this issue Nov 9, 2022 · 9 comments · May be fixed by #1096
Open

uuid of capture is missing in the "Captures" Module download #928

sebastiangaertner opened this issue Nov 9, 2022 · 9 comments · May be fixed by #1096
Assignees
Labels
good first issue Good for newcomers size: small Simple change tool: Captures Relates to the capture data tool (Captures page, formerly Trees) type: enhancement New feature or request

Comments

@sebastiangaertner
Copy link
Contributor

When choosing the available fields on the download option on the Captures module we have the token_uuid but not the capture_uuid. This needs to be available as well for API calls.

@sebastiangaertner sebastiangaertner added type: enhancement New feature or request good first issue Good for newcomers size: small Simple change tool: Captures Relates to the capture data tool (Captures page, formerly Trees) labels Nov 9, 2022
@sebastiangaertner
Copy link
Contributor Author

Token ID should be renamed to Token UUID and we need the additional Capture UUID added here in the list and it being available in the download.

image (2)

@sebastiangaertner
Copy link
Contributor Author

The Wallet UUID should also be available.

@elyselam
Copy link
Contributor

Please assign this to me

@nmcharlton
Copy link
Collaborator

@sebastiangaertner What's the reasoning for referring to these IDs as "UUID"?
UUID is just the type we use for the ID, and we should stick to the term ID unless there's a good reason to use UUID instead.

@sebastiangaertner
Copy link
Contributor Author

Oh I did not get this. Currently the uuid are not shown or? This is mainly to then interact with the wallet api straight out of the csv

@pranavkparti
Copy link
Contributor

The downloaded csv also has errors. The capture_id, grower_reference_id, and the image_url are downloaded incorrectly.
wallet_issue_928

@pranavkparti
Copy link
Contributor

  1. Wallet ID refers to the organization wallet ID, right?
  2. wallet refers to the grower's wallet, while wallet_name refers to the organization's wallet?

image

@ShimiRokach
Copy link

I solved the reference _id, grower_reference _id, image_url problem.
What was it about wallet id ? I don't see that you get such a field from the api

@mahimnamehta
Copy link

@sebastiangaertner To get more clear on this, from the image we are selecting Capture ID but we are not getting it , is it what you are refereeing to? uuid is a different unique if that is being generated for the captures? Please shed some light on me so we can be on the same page here.

I recently started so I am getting a bit comfortable with understanding the whole process. Thank you for your patience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers size: small Simple change tool: Captures Relates to the capture data tool (Captures page, formerly Trees) type: enhancement New feature or request
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

6 participants