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

BHBC-2085 #888

Merged
merged 27 commits into from
Dec 21, 2022
Merged

BHBC-2085 #888

merged 27 commits into from
Dec 21, 2022

Conversation

anissa-agahchen
Copy link
Contributor

@anissa-agahchen anissa-agahchen commented Dec 8, 2022

Overview

Links to Jira tickets

https://quartech.atlassian.net/browse/BHBC-2085

Description of relevant changes

The work in this ticket is based on the assumption that the darwin_core_source column in the occurrence_submission is populated with the result of the transform as a JSON object.

  • This code takes that object:
    ** finds all paths that contain a taxonID
    ** queries elastic search for the scientific and common name
    ** enriches the object at the same path with the scientific and common names
    ** returns the object

At a later point, this work will be merged with the transfom steps.

Note: look at tests to see how the object transformations

@anissa-agahchen anissa-agahchen added the Do Not Merge PR should not be merged label Dec 8, 2022
@codecov
Copy link

codecov bot commented Dec 8, 2022

Codecov Report

Merging #888 (6ddadae) into dev (9f44ff9) will increase coverage by 0.03%.
The diff coverage is 71.79%.

@@            Coverage Diff             @@
##              dev     #888      +/-   ##
==========================================
+ Coverage   62.17%   62.20%   +0.03%     
==========================================
  Files         379      380       +1     
  Lines       12248    12287      +39     
  Branches     2176     2183       +7     
==========================================
+ Hits         7615     7643      +28     
- Misses       4245     4250       +5     
- Partials      388      394       +6     
Impacted Files Coverage Δ
api/src/services/validation-service.ts 84.97% <ø> (ø)
...s/media/xlsx/transformation/xlsx-transformation.ts 3.70% <ø> (ø)
...p/src/features/surveys/view/SurveyObservations.tsx 37.83% <ø> (ø)
api/src/services/platform-service.ts 58.33% <20.00%> (-3.49%) ⬇️
api/src/services/taxonomy-service.ts 57.89% <50.00%> (-1.49%) ⬇️
api/src/services/dwc-service.ts 85.71% <85.71%> (ø)

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

@anissa-agahchen anissa-agahchen marked this pull request as ready for review December 12, 2022 22:17
@anissa-agahchen anissa-agahchen changed the title Bhbc 2085 BHBC-2085 Dec 12, 2022
@anissa-agahchen anissa-agahchen removed the Do Not Merge PR should not be merged label Dec 12, 2022
Copy link
Contributor

@AlfredRosenthal AlfredRosenthal left a comment

Choose a reason for hiding this comment

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

🐯

curtisupshall
curtisupshall previously approved these changes Dec 14, 2022
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.

I think it looks good! Just a handful of nitpicks re: comments and type defs

- It adds additional details, but may also lead to confusion if that taxon code isnt the issue.
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.

@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 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@NickPhura NickPhura merged commit 2693d63 into dev Dec 21, 2022
@NickPhura NickPhura deleted the BHBC-2085 branch December 21, 2022 22:35
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.

5 participants