Skip to content

Conversation

@annavik
Copy link
Member

@annavik annavik commented Jul 5, 2023

In this PR:

  • Add filtering feature
  • Add filtering links in various places
  • Add more info to views after API updates + tweak FE models
  • Extract gallery from occurrences and add it to sessions and species views as well

Screenshots:

One of the new gallery views:
Screenshot 2023-07-05 at 20 31 11

More info in some places, for example in occurrence detail view:
Screenshot 2023-07-05 at 18 59 35

Filtered collection (very basic UI for now):
Screenshot 2023-07-05 at 18 59 24

@netlify
Copy link

netlify bot commented Jul 5, 2023

Deploy Preview for ami-web canceled.

Name Link
🔨 Latest commit fb92d3f
🔍 Latest deploy log https://app.netlify.com/sites/ami-web/deploys/64a5d3060e21150008aaf7fb

@netlify
Copy link

netlify bot commented Jul 5, 2023

Deploy Preview for ami-storybook canceled.

Name Link
🔨 Latest commit fb92d3f
🔍 Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/64a5d3064e9f0f000800e0a4

@annavik annavik force-pushed the web-ui-filtering branch from 4079d31 to 763a0dd Compare July 5, 2023 18:40
@annavik annavik force-pushed the web-ui-filtering branch from 763a0dd to fc717f2 Compare July 5, 2023 18:56
Copy link
Collaborator

Choose a reason for hiding this comment

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

@annavik This is how the filtering params look for database models related with a foreignkey. They are autogenerated filters using a library, I can probably make custom aliases later

Copy link
Member Author

Choose a reason for hiding this comment

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

Great!!

Maybe labels below (those ones will be visible in the filter popup) can still be just "Deployment" and "Session", I guess that is the only thing users have to care about, or what do you think?

@mihow mihow merged commit 5723067 into main Jul 5, 2023
@mihow mihow deleted the web-ui-filtering branch August 14, 2023 23:34
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.

3 participants