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

SHAS Enquiry form #79

Merged
merged 1 commit into from
May 21, 2024
Merged

SHAS Enquiry form #79

merged 1 commit into from
May 21, 2024

Conversation

kyle1morel
Copy link
Collaborator

Description

Implements the form functionality for the new SHAS enquiry form. Allows full submission and draft saving. If enquiry is marked as related a new note will be created in the related submission. If enquiry is marked as wanting to apply, a new submission will automatically be created and added as related to the enquiry.

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@kyle1morel kyle1morel requested a review from wilwong89 as a code owner May 21, 2024 16:52
Copy link

github-actions bot commented May 21, 2024

Coverage Report (Frontend)

Totals Coverage
Statements: 31.99% ( 976 / 3051 )
Methods: 29.68% ( 179 / 603 )
Lines: 34.66% ( 573 / 1653 )
Branches: 28.18% ( 224 / 795 )

@slhurley
Copy link
Collaborator

I'm not sure if this is the right PR but reminder to yank the enquiry type from the Enquiry form but also to add the Enquiry text field to the Note that is created on the related submission.

@kyle1morel kyle1morel force-pushed the feature/enquiry branch 2 times, most recently from 545b0be to 9e3f976 Compare May 21, 2024 20:33
Copy link

github-actions bot commented May 21, 2024

Coverage Report (Application)

Totals Coverage
Statements: 47.98% ( 699 / 1457 )
Methods: 38.27% ( 106 / 277 )
Lines: 60.81% ( 467 / 768 )
Branches: 30.58% ( 126 / 412 )

app/src/routes/v1/enquiry.ts Show resolved Hide resolved
app/src/controllers/submission.ts Outdated Show resolved Hide resolved
app/src/services/activity.ts Outdated Show resolved Hide resolved
app/src/db/migrations/20240506000000_004-shas-intake.ts Outdated Show resolved Hide resolved
app/src/db/migrations/20240506000000_004-shas-intake.ts Outdated Show resolved Hide resolved
Copy link

codeclimate bot commented May 21, 2024

Code Climate has analyzed commit 56d6e60 and detected 43 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 8
Duplication 35

The test coverage on the diff in this pull request is 54.3% (50% is the threshold).

This pull request will bring the total coverage in the repository to 43.0% (5.5% change).

View more on Code Climate.

frontend/src/utils/utils.ts Show resolved Hide resolved
app/src/services/enquiry.ts Show resolved Hide resolved
app/src/routes/v1/enquiry.ts Show resolved Hide resolved
app/src/routes/v1/enquiry.ts Show resolved Hide resolved
@kyle1morel kyle1morel changed the base branch from master to feature/intake-form May 21, 2024 23:23
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 5882 lines exceeds the maximum allowed for the inline comments feature.

@kyle1morel kyle1morel merged commit 70e570e into feature/intake-form May 21, 2024
16 of 17 checks passed
@kyle1morel kyle1morel deleted the feature/enquiry branch May 21, 2024 23:53
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.

3 participants