-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat: seeds button uptake #3677
Conversation
✅ Deploy Preview for bloom-demo-public ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are accessibility issues in these changes.
sites/partners/src/components/applications/PaperApplicationForm/sections/FormDemographics.tsx
Outdated
Show resolved
Hide resolved
sites/partners/src/components/applications/PaperApplicationForm/FormMember.tsx
Outdated
Show resolved
Hide resolved
❌ Deploy Preview for bloom-partners-listings-approval failed.
|
@jaredcwhite one way we can solve the "jest-shared-helpers" failing test is to change the tsconfig.json in the shared-helpers to have this line |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found so far, but two notes:
- There is at least one usage of the ui-components button in a test file -
StatusBar.test.tsx
- The failing cypress tests appear to be failing because the TextArea component is unable to be typed in by Cypress. Don't know why that would be happening with your changes. I'll continue to investigate it today
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me so I'm approving it. The error that is happening for the textarea is because apparently it is partially covered by the save button. But I don't know why cypress thinks that. Here is the error:
I am able to get past this by changing the type step to this. But we should probably look into why Cypress thinks the button is covering the textarea
cy.getByTestId("preference-description").type("Preference Description", {
force: true,
})
Thanks @ludtkemorgan for looking into that! Weird stuff. We'll have to keep an eye on it. |
Hmm, yeah we might just have to see if we can patch that in Seeds directly. |
@jaredcwhite Any luck on the tests here? |
@emilyjablonski I haven't spent any more time on it. Still stumped. 🤷🏻♂️ |
Success! I was able to fix the issues with the Cypress partners tests (and also unit tests). The one remaining failing test was the issue we already identified which requires a fix in Seeds. (The other backend failure seems to be a fluke…a previous test run was successful.) |
* feat: seeds button uptake (bloom-housing#3677) * fix: validate radius geocoding preferences (bloom-housing#3718) * fix: validate radius geocoding preferences * fix: move to more generic * fix: add test * fix: one more test * fix: change to true/false * feat: add address verification for preferences step (bloom-housing#3715) * feat: add collectAddress checkbox with subfields * test: update preference tests * fix: add minimum value validation for radius field * fix: make collect address not required * fix: expand collect address fields * fix: change fields order in PreferenceDrawer * feat: add address holder fields to application * fix: make added field optional * fix: display errors properly * feat: add address holder fields to application summary * feat: add address verification for preferences step * feat: adjust padding for application summary * fix: move address holder name and relationship fields to extraData * fix: remove redundant backend address holder fields * fix: use enum for address holder fields * fix: add alternate address form component * fix: remove redundant fields from new address form * fix: verify preferences address when collectAddress true * fix: block going back on address verification add string to translation * fix: use onClick to block address Verification back button * fix: update dependencies for backend (bloom-housing#3694) * feat: core changes for application style alignment (bloom-housing#3662) * test: fix cypress lat long error (bloom-housing#3746) * fix: padding issue on address verify (bloom-housing#3745) * feat: add show mandated accounts flag (bloom-housing#3773) * feat: mA-3752 feat: add SHOW_MANDATED_ACCOUNTS flag * fix: add feature flag to bloom public next config * fix: remove flag from partners * feat: add address holder fields to paper application (bloom-housing#3716) * feat: add collectAddress checkbox with subfields * test: update preference tests * fix: add minimum value validation for radius field * fix: make collect address not required * fix: expand collect address fields * fix: change fields order in PreferenceDrawer * feat: add address holder fields to application * fix: make added field optional * fix: display errors properly * feat: add address holder fields to application summary * feat: adjust padding for application summary * feat: add address holder fields to paper application * fix: use enum for address holder paper application fields * fix: remove redundant backend fields * fix: add correct labels, and values for extraData summary * fix: export geocoding values enum from backend * fix: display address as last extraData field * fix: remove placeholders from form * fix: set default values after listingDto fetched * style: adjust details multiselect question * feat: add geocoding preference label (bloom-housing#3765) * feat: add additional fields info to add listing preferences * feat: add additional fields column to table * refactor: reuse additional field tag * refactor: change base tag style * fix: revert onClose drawer function * fix: add geocoding fields to csv (bloom-housing#3778) * fix: update seeds button (bloom-housing#3793) * fix: add translations for geocoding (bloom-housing#3784) * fix: add translations for geocoding * fix: capitalize translation for qualifying address in vi.json * fix: provides additional fields info text overflows (bloom-housing#3809) * fix: provides additional fields info text overflows * fix: check if length of links greater than 0 * refactor: remove console.log * fix: add links to preference description (bloom-housing#3813) * fix: seeds button layout fixes for Partners Application sidebar (bloom-housing#3789) * feat: geocoding preference set up fixed layers (bloom-housing#3808) * feat: add geocoding map layers to multiselect option * fix: remove link from select description * fix: default value for mapLayerId * fix: add decorators for mapLayers dto's and entities * fix: use smaller font for mapLayer select helper text * fix: test fixes * fix: add map layer geocoding checks (bloom-housing#3825) * fix: add map layer geocoding checks * fix: review comment fix --------- Co-authored-by: Krzysztof Zięcina <kziecina@airnauts.com> Co-authored-by: Emily Jablonski <65367387+emilyjablonski@users.noreply.github.com> Co-authored-by: cade-exygy <131277283+cade-exygy@users.noreply.github.com> Co-authored-by: ColinBuyck <53269332+ColinBuyck@users.noreply.github.com> Co-authored-by: Jared White <jared@jaredwhite.com>
Issue Overview
This PR addresses #3595
Description
This PR swaps out all UIC buttons with Seeds buttons in the Core applications. Note that this doesn't address any buttons which may be embedded within other UIC components.
How Can This Be Tested/Reviewed?
Visit pages in the Public and Partners sites. Ensure that the buttons work and aren't broken. I noticed some translations have odd capitalization, which means you can now tell (because the old buttons were ALL CAPPS), but I would recommend addressing those in followup PRs.
NOTE: there are two Cypress tests for Partners which I cannot figure how to resolve. The error messages are confusing and don't make sense. Also note: the jest-shared-helpers test is failing, but that's due to an apparent TypeScript issue coming from Seeds, so if we want to resolve that we'll need to address in a separate issue/PR in that repo.
Checklist:
yarn generate:client
and/or created a migration if I made backend changes that require themReviewer Notes:
Steps to review a PR:
On Merge:
If you have one commit and message, squash. If you need each message to be applied, rebase and merge.