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

New SHAS intake skeleton #73

Closed
wants to merge 22 commits into from

Conversation

kyle1morel
Copy link
Collaborator

@kyle1morel kyle1morel commented May 1, 2024

Description

Implements the skeleton + basic form functionality for the new intake form. This form is designed to eventually fully replace the existing CHEFS SHAS intake form. Advanced functionality such as 3rd party API lookups, mapping, document upload, etc., is missing and scheduled to be done in the future.

58babb3 Update + Add unit tests
124850f Intake submit & confirmation
ea48354 Add fourth step - Permits
7981cbd Add third step - Location
815d192 Added second step - Housing
08f70bb Finalized styling. Added custom Stepper header and navigation components
aa923d4 New intake form - styling & first step

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

Copy link

github-actions bot commented May 1, 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 )

@kyle1morel kyle1morel force-pushed the feature/intake-form branch from fe25e98 to 5e4510c Compare May 1, 2024 19:54
Copy link

github-actions bot commented May 1, 2024

Coverage Report (Frontend)

Totals Coverage
Statements: 31.97% ( 976 / 3053 )
Methods: 29.64% ( 179 / 604 )
Lines: 34.64% ( 573 / 1654 )
Branches: 28.18% ( 224 / 795 )

frontend/src/utils/utils.ts Show resolved Hide resolved
frontend/src/components/intake/ShasIntakeForm.vue Outdated Show resolved Hide resolved
frontend/src/components/intake/ShasIntakeForm.vue Outdated Show resolved Hide resolved
frontend/src/components/intake/ShasIntakeForm.vue Outdated Show resolved Hide resolved
Copy link

codeclimate bot commented May 1, 2024

Code Climate has analyzed commit 58babb3 and detected 13 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 6
Duplication 4
Bug Risk 3

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

This pull request will bring the total coverage in the repository to 36.0% (-1.7% change).

View more on Code Climate.

@kyle1morel kyle1morel force-pushed the feature/intake-form branch 3 times, most recently from 7024312 to 960c7b5 Compare May 7, 2024 16:57
app/src/controllers/submission.ts Outdated Show resolved Hide resolved
app/src/services/activity.ts Show resolved Hide resolved
app/src/controllers/submission.ts Outdated Show resolved Hide resolved
@kyle1morel kyle1morel force-pushed the feature/intake-form branch from 960c7b5 to 124850f Compare May 7, 2024 23:56
@kyle1morel kyle1morel closed this May 8, 2024
@kyle1morel kyle1morel reopened this May 8, 2024
@kyle1morel kyle1morel changed the title DRAFT: New SHAS intake form New SHAS intake form May 8, 2024
@kyle1morel kyle1morel changed the title New SHAS intake form New SHAS intake skeleton May 8, 2024
@kyle1morel kyle1morel marked this pull request as ready for review May 8, 2024 19:23
@kyle1morel kyle1morel requested a review from wilwong89 as a code owner May 8, 2024 19:23
@kyle1morel kyle1morel requested a review from sanjaytkbabu May 8, 2024 19:24
@kyle1morel kyle1morel changed the base branch from master to feature/shas-intake-form May 8, 2024 21:26
@wilwong89
Copy link
Contributor

SubmissionForm.vue is still using the boolean checkboxes for fields like financiallySupported and financiallySupportedBC that were converted to Yes/No/Unsure. Can't save the submission if those converted fields are checked off.

@slhurley
Copy link
Collaborator

slhurley commented May 8, 2024

  • Need help text outside of the field entry on the comma-separated PIDs field (or consistent help text/hints on fields)
  • When entering the "Search the address" field it leaves it there shouldn't it disappear after it does the search (presumably this is coming with the validation layer)
  • What is Locality? Shouldn't it say City? (I could be wrong but never heard of this)
  • "Provide a coordinate" has the word coordinate spelled wrong
  • Can we make sure validation also chokes if you don't enter BC in the province? Or perhaps just don't let them change this field?
  • Spatial file is misspelled
  • Need to get missing hints and missing X business days on the confirmation from Subin
  • I think we need to remove the New vs. Applied permit status. Either a proponent has applied or they haven't.

@Subin1Doo
Copy link

UI updates:
Housing tab

  • in first section "Provide additional information" - change the font and color - same as all the other - not bolded

  • still missing the upload document design that goes under provide additional information text box.

  • type of residential multi select check boxes - when checked off should be the same color as radio buttons?

  • get rid of "?" next to Is this project being financially supported by any of the following

  • BC Housing - "?" bubble should say: BC Housing welcomes the opportunity to work with individuals and organizations to create affordable housing solutions.

  • the word BC Housing in text bubble - hyperlinked to https://www.bchousing.org/proejcts-partners/partner-with-us

  • "?" next to BC Housing not centre aligned, move up a little bit

  • Change the font color for BC Housing, Indigenous Housing Provider, Non-profit housing society, Housing co-operative, so it's the not the same color as yes/no/unsure.

Location tab

  • get rid of "?" next to provide additional location details

@Subin1Doo
Copy link

Confirmation page message change to: Your submission will be reviewed by a Housing Navigator. You may be contacted if needed. Please check your email for the confirmation email and keep the confirmation ID for future reference.

@Subin1Doo
Copy link

For this: type of residential multi select check boxes - when checked off should be the same color as radio buttons?
I meant to change the color of the multi-select check boxes to that light sky blue-ish we were using for all radio buttons.
Can we change it back to that color for all radio/ multi-select buttons?

The question mark next to "provide additional information" isn't centre-aligned

for the BC housing hyperlink - can we hyperlink the word BC Housing on the actual form then? not the one inside the tooltip text.

Thank you!

@kyle1morel kyle1morel force-pushed the feature/intake-form branch from 43207c8 to 6252828 Compare May 9, 2024 23:23
@slhurley
Copy link
Collaborator

  • We need the other hyperlinks on the other financially supported things like we did on BC Housing
  • There is also a link for the Geomark service: https://apps.gov.bc.ca/pub/geomark/overview
  • Can Province default to BC
  • If you add the name of something to the text that pops up after you select Yes to one or more of the financially supported things and then go back and change your mind and click "No to All", it doesn't clear the text fields. It looks like it does because the field goes to hidden but if you go back and click Yes again the text filled out reappears...maybe ok as long as it isn't saving if they stick to the "No".

Minor things!

@Subin1Doo
Copy link

Hyperlinks:
Indigenous housing provider:
https://www.bchousing.org/housing-assistance/rental-housing/indigenous-housing-providers
Non-profit housing society:
https://bcnpha.ca/member-programs-list/
housing co-operative:
https://www.chf.bc.ca/find-co-op/

For spatial file upload
"see acceptable file formats":
https://portal.nrs.gov.bc.ca/documents/10184/0/SpatialFileFormats.pdf/39b29b91-d2a7-b8d1-af1b-7216f8db38b4

One last thing..
For "Select all provincially issued permits you think you might need (optional)"
Can we make the length of the drop down for permits longer so it's the same as how it is for "Have you applied for any provincial permits for this project?"section.

Thank you!!

@kyle1morel kyle1morel force-pushed the feature/intake-form branch from 6252828 to 65d14a6 Compare May 13, 2024 22:38
@Subin1Doo
Copy link

Hey Kyle,
looks like we have to had another question / section to the form.
On the location tab - at the very top of the page add a new section
"Has the location of the project been affected by natural disaster?"
Yes / no radio button

Also, based on everyone's feedback we want to move the notice of collection message to the first page of the form, top of basic info tab. Check link below to see update on this:
https://www.figma.com/design/MOKpr9Jfu4ptlATQ23lFUf/SHAS?node-id=597%3A3896&t=xloJnti3d5zuClA9-1

Thank you!

@kyle1morel kyle1morel force-pushed the feature/intake-form branch from 65d14a6 to 6d286aa Compare May 14, 2024 18:47
@Subin1Doo
Copy link

Reminder to add labels to all dropdown/ type fields for accessibility. Thank you!

@Subin1Doo
Copy link

In the Location tab, below the street address / coordinates section, add another section labeled as "Is there anything else you would like to tell us about this projects location?" with a free text box. This section is optional.

Thank you!

kyle1morel and others added 22 commits May 21, 2024 15:44
Allow multiple files to be selected for upload at once
Removed col class from form components
Migration included: data type changes, many new columns
Bug noticed by navs where backspacing caused the whitespace to be removed automatically
Slipstream update
…eForm

Update PrimeVue to 3.52.0
Rebase to latest ShasIntakeForm.vue changes
Integrate multiple WIP intake form validation rules
Additional validation error message handling
@kyle1morel kyle1morel force-pushed the feature/intake-form branch from 70e570e to 25a2f84 Compare May 21, 2024 23:58
@kyle1morel kyle1morel closed this May 22, 2024
@kyle1morel kyle1morel deleted the feature/intake-form branch May 22, 2024 00:00
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