Skip to content

Conversation

@KjartanE
Copy link
Contributor

Links to Jira Tickets

Description of Changes

  • Adjust species selector to follow similar style as select user.
  • Update components for front-end.
  • Requires possible update due to itis changes.

Testing Notes

  • {List any relevant testing considerations, necessary pre-reqs, and areas of the app to focus on. Specifically, include anything that will help the reviewers of this PR verify the code is functioning as expected.}

@github-actions
Copy link

@codecov
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

Attention: 107 lines in your changes are missing coverage. Please review.

Comparison is base (66db344) 57.25% compared to head (e0fc577) 56.98%.

Files Patch % Lines
...ts/species/components/SpeciesAutocompleteField.tsx 2.77% 35 Missing ⚠️
...c/components/species/AncillarySpeciesComponent.tsx 5.55% 17 Missing ⚠️
...p/src/components/species/FocalSpeciesComponent.tsx 5.55% 17 Missing ⚠️
app/src/hooks/itis/useItisApi.ts 8.33% 11 Missing ⚠️
api/src/services/platform-service.ts 22.22% 7 Missing ⚠️
.../components/species/components/SelectedSpecies.tsx 16.66% 5 Missing ⚠️
...ponents/species/components/SpeciesSelectedCard.tsx 20.00% 4 Missing ⚠️
app/src/hooks/useContext.tsx 20.00% 4 Missing ⚠️
api/src/models/survey-view.ts 75.00% 0 Missing and 2 partials ⚠️
.../src/components/species/components/SpeciesCard.tsx 33.33% 2 Missing ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1201      +/-   ##
==========================================
- Coverage   57.25%   56.98%   -0.28%     
==========================================
  Files         586      594       +8     
  Lines       17877    17983     +106     
  Branches     2745     2768      +23     
==========================================
+ Hits        10235    10247      +12     
- Misses       6978     7070      +92     
- Partials      664      666       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@NickPhura NickPhura changed the title SIMSBIOHUB-453: Add Attributes to Species Observations SIMSBIOHUB-453: Update Species Menus Jan 24, 2024
@KjartanE KjartanE marked this pull request as ready for review January 25, 2024 23:23
@github-actions
Copy link

@KjartanE KjartanE added the Ready For Review PR is ready for review label Jan 25, 2024
@NickPhura
Copy link
Contributor

NickPhura commented Jan 29, 2024

@KjartanE
Make sure to add any env vars to the env.docker (as well as the docker-compose and .pipeline config file)

Also as we discussed, we can just call SOLR directly from the frontend, rather than proxying it through the API. Nothing about the SOLR endpoint, thankfully, is private or protected.

curtisupshall and others added 9 commits February 1, 2024 14:29
…rations and Telemetry (#1198)

* Add support for hiding/showing table columns for Obserations and Telemetry
* Added sessionStorage call in order to cache column visibility across app reloads
* Renamed ObservationComponent and ManualTelemetryComponent to ObservationsTableContainer and ManualTelemetryTableContainer, respectively
* Adding PageHeader component to all relevant pages

* Small Layout Fix

* Fixing test

* Remove uuid call from SkeletonLoaders.tsx

* Lint fixes.

* ignore-skip

---------

Co-authored-by: Curtis Upshall <[email protected]>
Co-authored-by: Nick Phura <[email protected]>
…nimals (#1202)

* Combined the Telemetry and Observations map/table view on the Survey page, allowing users to toggle between them;
* Added server-side support for paginating the observations fetching endpoint;
* Modified the Manage Observations page to use server-side pagination;
* Added a new endpoint to fetch all the spatial data relating to survey observations;
* Deleted some old code related to Survey Summary Results.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

1 Security Hotspot
5.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Contributor

@curtisupshall curtisupshall left a comment

Choose a reason for hiding this comment

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

Code looks good to me!

@KjartanE KjartanE merged commit 6118b56 into dev Feb 5, 2024
@KjartanE KjartanE deleted the SIMSBIOHUB-453 branch February 5, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready For Review PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants