Skip to content

Conversation

mihow
Copy link
Collaborator

@mihow mihow commented Sep 17, 2024

Add the hierarchical taxa search to the Species tab. This uses the same component as the Occurrences tab. I updated the url param to be taxon for both placements. Even though they mean slightly different things in both contexts.

image

Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for ami-storybook ready!

Name Link
🔨 Latest commit 86f64b2
🔍 Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/66e8c7c19e9d5c0008282bb9
😎 Deploy Preview https://deploy-preview-578--ami-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for ami-dev ready!

Name Link
🔨 Latest commit 86f64b2
🔍 Latest deploy log https://app.netlify.com/sites/ami-dev/deploys/66e8c7c143bb830008d7765a
😎 Deploy Preview https://deploy-preview-578--ami-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 26
Accessibility: 89
Best Practices: 92
SEO: 100
PWA: 80
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@mihow mihow requested a review from annavik September 17, 2024 00:16
@mihow mihow marked this pull request as ready for review September 17, 2024 00:22
to={getAppRoute({
to: APP_ROUTES.OCCURRENCES({ projectId }),
filters: { determination: item.id },
filters: { taxon: item.id },
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious, did you notice this had to be updated thanks to build errors? That would be my hope!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case I was searching the codebase, but I'll run the tests first next time to try that!

Copy link
Member

@annavik annavik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this! Also the new URL param key makes more sense 👌

@mihow mihow merged commit 3e0f827 into main Sep 17, 2024
6 checks passed
@mihow mihow deleted the feat/post-demo-fixes branch September 17, 2024 16:31
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

Successfully merging this pull request may close these issues.

2 participants