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-276: Remove Lead Biologist from survey edit and create page #1108

Merged
merged 9 commits into from
Oct 5, 2023

Conversation

curtisupshall
Copy link
Contributor

@curtisupshall curtisupshall commented Oct 5, 2023

Links to Jira Tickets

Description of Changes

  • Removes survey lead biologist from frontend and backend code
  • Adds a migration to fix the survey delete procedure
  • Fixes issues with the Observations Table

Testing Notes

You should be able to:

  • Create a survey
  • Edit a survey
  • Ddelete a survey

@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #1108 (9961285) into dev (73681f3) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev    #1108      +/-   ##
==========================================
- Coverage   63.30%   63.29%   -0.02%     
==========================================
  Files         531      531              
  Lines       15786    15781       -5     
  Branches     2445     2439       -6     
==========================================
- Hits         9994     9988       -6     
- Misses       5107     5108       +1     
  Partials      685      685              
Files Coverage Δ
api/src/models/survey-create.ts 77.96% <ø> (-0.73%) ⬇️
api/src/models/survey-update.ts 80.64% <ø> (-0.61%) ⬇️
api/src/models/survey-view.ts 82.85% <ø> (-0.48%) ⬇️
api/src/paths/project/{projectId}/survey/create.ts 96.15% <ø> (ø)
api/src/paths/project/{projectId}/survey/list.ts 96.66% <ø> (ø)
...hs/project/{projectId}/survey/{surveyId}/update.ts 96.29% <ø> (ø)
...roject/{projectId}/survey/{surveyId}/update/get.ts 84.37% <ø> (ø)
...aths/project/{projectId}/survey/{surveyId}/view.ts 95.83% <ø> (ø)
api/src/repositories/survey-repository.ts 96.77% <ø> (ø)
...ures/surveys/components/GeneralInformationForm.tsx 20.83% <ø> (ø)
... and 5 more

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

@curtisupshall curtisupshall marked this pull request as ready for review October 5, 2023 16:16
al-rosenthal
al-rosenthal previously approved these changes Oct 5, 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 everything works 🛠️

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 5, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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
8.3% 8.3% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

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. Everything is working as expected on my local 🐸

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.

🐕

@al-rosenthal al-rosenthal merged commit d4aa9c4 into dev Oct 5, 2023
@al-rosenthal al-rosenthal deleted the SIMSBIOHUB-276 branch October 5, 2023 18:26
KjartanE pushed a commit that referenced this pull request Oct 6, 2023
#1108)

Removed lead biologist from survey edit and create page
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.

4 participants