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

Organization filter in tree verification not updating #73

Closed
sebastiangaertner opened this issue Apr 20, 2021 · 1 comment · Fixed by #87
Closed

Organization filter in tree verification not updating #73

sebastiangaertner opened this issue Apr 20, 2021 · 1 comment · Fixed by #87
Assignees
Labels
good first issue Good for newcomers size: small Simple change tool: Verify Relates to the capture verification tool (Verify page) type: bug Something isn't working

Comments

@sebastiangaertner
Copy link
Contributor

When i change an entity type in the DB from p to O or o to O or anything else it does not seem to reflect the filter. Can we only show capital O types?

@nmcharlton nmcharlton transferred this issue from Greenstand/treetracker-admin-api Apr 20, 2021
@nmcharlton nmcharlton added type: bug Something isn't working tool: Verify Relates to the capture verification tool (Verify page) size: small Simple change labels Apr 20, 2021
@nmcharlton
Copy link
Collaborator

The simplest fix is to update the following API call to use [type]=O:

const query = `${process.env.REACT_APP_API_ROOT}/api/organizations?filter[where][type]=o`;

Note that there are other associated organization filter issues that may move this logic to the API side (#26, #37, #70).

@nmcharlton nmcharlton added the good first issue Good for newcomers label Apr 20, 2021
@gwynndp gwynndp self-assigned this May 16, 2021
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: Verify Relates to the capture verification tool (Verify page) type: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants