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

SIMSBIOHUB 347/341 - Names from shapefiles #1146

Merged
merged 8 commits into from
Oct 31, 2023

Conversation

GrahamS-Quartech
Copy link
Contributor

Links to Jira Tickets

Description of Changes

  • Added some additional logic to the API to extract a name and description from the provided shape file when creating sampling sites.
  • It will search the keys of the geometry's properties for "name" or "label" and "desc" or "description" (exact match, should we allow substring?), then use the value for the sampling site name and description respectively.
  • Added a character limit to the sampling site name field in the edit form.

Testing Notes

  • OminecaShapefile.zip's properties are an example of something that won't be picked up by this approach.
  • 62_Features_Shapefile.zip properties will be used by this approach.

@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #1146 (c1cbe05) into dev-staging (3b1400a) will decrease coverage by 0.02%.
Report is 3 commits behind head on dev-staging.
The diff coverage is 41.66%.

@@               Coverage Diff               @@
##           dev-staging    #1146      +/-   ##
===============================================
- Coverage        62.13%   62.11%   -0.02%     
===============================================
  Files              554      554              
  Lines            16914    16940      +26     
  Branches          2598     2605       +7     
===============================================
+ Hits             10510    10523      +13     
- Misses            5697     5705       +8     
- Partials           707      712       +5     
Files Coverage Δ
...pling-sites/edit/components/SampleSiteEditForm.tsx 0.00% <ø> (ø)
...it/components/SampleSiteGeneralInformationForm.tsx 0.00% <ø> (ø)
api/src/repositories/sample-location-repository.ts 90.24% <50.00%> (-2.07%) ⬇️
api/src/services/sample-location-service.ts 83.78% <40.00%> (-16.22%) ⬇️

... and 9 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@GrahamS-Quartech GrahamS-Quartech changed the title SIMSBIOHUB 347/341 SIMSBIOHUB 347/341 - Names from shapefiles Oct 26, 2023
Copy link
Contributor

@al-rosenthal al-rosenthal left a comment

Choose a reason for hiding this comment

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

ran this locally and it works as expected

@curtisupshall curtisupshall added the Staging Will merge into staging branch label Oct 27, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@JeremyQuartech JeremyQuartech 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, tested changes locally and everything worked as described 👍

@GrahamS-Quartech GrahamS-Quartech merged commit ac199ba into dev-staging Oct 31, 2023
16 of 20 checks passed
@GrahamS-Quartech GrahamS-Quartech deleted the SIMSBIOHUB-347 branch October 31, 2023 19:11
KjartanE pushed a commit that referenced this pull request Nov 6, 2023
* Added some additional logic to the API to extract a name and description from the provided shape file when creating sampling sites.
* It will search the keys of the geometry's properties for "name" or "label" and "des" or "desc" or "descr" (exact match), then use the value for the sampling site name and description respectively.
* Added a character limit to the sampling site name field in the edit form.

---------

Co-authored-by: JeremyQuartech <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Staging Will merge into staging branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants