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

Make all capture IDs (tree IDs) and planter IDs links to webmap #50

Closed
nmcharlton opened this issue Mar 26, 2021 · 6 comments · Fixed by #79
Closed

Make all capture IDs (tree IDs) and planter IDs links to webmap #50

nmcharlton opened this issue Mar 26, 2021 · 6 comments · Fixed by #79
Assignees
Labels
size: medium Medium-size issue tool: Captures Relates to the capture data tool (Captures page, formerly Trees) tool: Growers Relates to the Grower Management tool (Growers page, formerly Planters) 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

Everywhere we display a capture ID (currently referred to as tree ID), we should make the text a link to the web map displaying that tree:

`${process.env.REACT_APP_WEBMAP_DOMAIN}/treeid=${id}`

Similarly, where we display the planter ID, we should link to the webmap displaying all of the planter's trees:

`${process.env.REACT_APP_WEBMAP_DOMAIN}/userid=${id}`

These IDs are displayed in the following places:

  • Tree detail (Verify page)
  • Planters page
  • Planter detail (Planters page and Verify page)
  • Trees page

The user should see some indication that clicking the link will open the webmap, and the link should always open in a new tab.

@nmcharlton
Copy link
Collaborator Author

@sebastiangaertner Does this seem like an appropriate solution?

@nmcharlton nmcharlton added tool: Captures Relates to the capture data tool (Captures page, formerly Trees) tool: Verify Relates to the capture verification tool (Verify page) size: medium Medium-size issue tool: Growers Relates to the Grower Management tool (Growers page, formerly Planters) UI/UX Requires some UI/UX design input labels Apr 10, 2021
@VLuisa
Copy link
Contributor

VLuisa commented May 3, 2021

I can take a look at this one!

@VLuisa
Copy link
Contributor

VLuisa commented May 4, 2021

@nmcharlton Do you think something like this suits the need? Here, the ID number and 'Launch in New' icon are all clickable as a Link. Open to suggestions! (If this looks ok I'll go ahead and apply it to the other places mentioned)

image

@nmcharlton
Copy link
Collaborator Author

@VLuisa That looks awesome.
I've shared it on Slack to get some opinions from our tame users.

@VLuisa
Copy link
Contributor

VLuisa commented May 5, 2021

@nmcharlton Thanks for sharing it there! I followed up with one another question there so leaving the PR as a draft until I hear back

@VLuisa
Copy link
Contributor

VLuisa commented May 5, 2021

Just heard back, so I'll leave it as it is!

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: Growers Relates to the Grower Management tool (Growers page, formerly Planters) 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
None yet
Development

Successfully merging a pull request may close this issue.

2 participants