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.

Sign up for GitHub

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

Jump to bottom

Release Test to Prod #1311

Merged
merged 11 commits into from
Jun 26, 2024
Merged

Release Test to Prod #1311

merged 11 commits into from
Jun 26, 2024

Conversation

mauberti-bc
Copy link
Collaborator

No description provided.

mauberti-bc and others added 7 commits June 12, 2024 16:48
@mauberti-bc
SIMSBIOHUB-579/583: Updated Animal Workflow (Capture, Mortality, Meas…
34cffde 
…urements, Profile) (#1291)

New capture, moratality and measurement pages for 'Manage Animals'.
Using Critterbase bulk endpoints.
@MacQSL
SIMSBIOHUB-588 Capture Method / Date Time (#1305)
d71f5f7 
Updated capture_timestamp / release_timestamp -> capture_date / capture_time etc
@MacQSL
Merge pull request #1306 from bcgov/dev
28e1356 
Release Dev to Test
Updated Animal Workflow
@mauberti-bc @NickPhura
Text input for capture and mortality locations (#1308)
cfbc84f 
Add: Added input textfield boxes to manually enter capture and mortality latitude/longitude values. Updated validation of coordinates
Bug fix: remove overflow hidden from the telemetry deployments list

---------

Co-authored-by: Nick Phura <[email protected]>
@mauberti-bc @NickPhura
SIMSBIOHUB-595: Remove project dates and programs (#1309)
a3cb8a7 
* remove project dates and programs
* Remove remaining references to program.
* Drop program code table.
---------

Co-authored-by: Nick Phura <[email protected]>
@mauberti-bc @NickPhura
SIMSBIOHUB-593: Add constraint for a user to have only one role per P…
e4a401e 
…roject (#1307)

* Add database constraint to enforce one role per user per project
* Add project member role icons to form control

---------

Co-authored-by: Nick Phura <[email protected]>
@mauberti-bc
Merge pull request #1310 from bcgov/dev
8fe4da3 
Release Dev to Test
@codecov Codecov
Copy link

codecov bot commented Jun 24, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 9.68407% with 1315 lines in your changes missing coverage. Please review.

Project coverage is 49.21%. Comparing base (3b04e0f) to head (56bf843).
Report is 2 commits behind head on prod.

Files Patch % Lines
...ile/captures/capture-form/edit/EditCapturePage.tsx 0.00% 81 Missing ⚠️
...tures/surveys/animals/list/AnimalListContainer.tsx 0.00% 76 Missing ⚠️
.../components/location/CaptureLocationMapControl.tsx 0.00% 72 Missing ⚠️
...omponents/location/MortalityLocationMapControl.tsx 0.00% 72 Missing ⚠️
...ortality/mortality-form/edit/EditMortalityPage.tsx 0.00% 71 Missing ⚠️
...captures/capture-form/create/CreateCapturePage.tsx 0.00% 60 Missing ⚠️
...urveys/animals/animal-form/edit/EditAnimalPage.tsx 0.00% 55 Missing ⚠️
...lity/mortality-form/create/CreateMortalityPage.tsx 0.00% 55 Missing ⚠️
.../surveys/animals/profile/markings/MarkingsForm.tsx 0.00% 47 Missing ⚠️
...ys/animals/animal-form/create/CreateAnimalPage.tsx 0.00% 41 Missing ⚠️
... and 93 more
Additional details and impacted files 
@@            Coverage Diff             @@
##             prod    #1311      +/-   ##
==========================================
- Coverage   52.65%   49.21%   -3.45%     
==========================================
  Files         581      639      +58     
  Lines       16757    17707     +950     
  Branches     2600     2787     +187     
==========================================
- Hits         8823     8714     -109     
- Misses       7344     8409    +1065     
+ Partials      590      584       -6

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

NickPhura added 2 commits June 25, 2024 11:50
@NickPhura
BugFixes: Bug Fixes and Misc Cleanup (#1312)
4113630 
* Migrate deprecated attachment file_type values (was impacting Test and Prod).
* Remove deprecated wldtaxonomic_units_id and field_method_id columns.
* Update minimum replicas to stop some of the platform alert spam about horizontal pod autoscalars
* Comment out HorizontalPodAutoscaler in API/APP
* Update clamav readme and files.
@NickPhura
Release Dev to Test - Merge pull request #1313 from bcgov/dev
2e6cc66 
Release Dev to Test
@NickPhura
Copy link
Collaborator

NickPhura commented Jun 25, 2024 •
edited
Loading

The BioHub taxon common names fix has been released to Test/Prod.

Did a quick test in SIMS Test: The taxons with no common names are returning properly now when editing a survey.

NickPhura
NickPhura approved these changes Jun 25, 2024
View reviewed changes
Copy link
Collaborator

@NickPhura NickPhura left a comment

Choose a reason for hiding this comment

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

🔥

@NickPhura NickPhura self-requested a review June 25, 2024 22:29
NickPhura added 2 commits June 25, 2024 16:18
@NickPhura
BugFix: Fix migration to drop missed instance of wldtaxonomic_units_i…
ce03ca4 
…d column (#1314)

* Fix migration to drop missed instance of wldtaxonomic_units_id column
* Disable PR builds when merging Dev to Test
@NickPhura
Release Dev to Test - Merge pull request #1315 from bcgov/dev
56bf843 
Release Dev to Test
NickPhura
NickPhura approved these changes Jun 26, 2024
View reviewed changes
Copy link
Collaborator

@NickPhura NickPhura left a comment

Choose a reason for hiding this comment

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

🔥 🔥

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
11.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@NickPhura NickPhura merged commit a7a7ff6 into prod Jun 26, 2024
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NickPhura NickPhura NickPhura approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mauberti-bc @NickPhura @MacQSL