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

EP-2564 - minimize EFT agreement #1167

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wrandall22
Copy link
Contributor

Jira

This PR puts the EFT agreement behind a link to reduce friction and vertical space on the bank account creation form.

@wrandall22 wrandall22 force-pushed the EP-2564-minimize-eft-agreement branch from 356f2d5 to 35445c2 Compare March 11, 2025 21:34
@wrandall22 wrandall22 marked this pull request as ready for review March 12, 2025 14:39
@wrandall22
Copy link
Contributor Author

wrandall22 commented Mar 12, 2025

I see potential for needing to make the checkbox text no longer all caps as well as maybe changing the reference within the agreement to the checkbox while UAT is happening, we will see what the stakeholders want. I could also see potential for needing to move the agreement below the checkbox.

@wrandall22 wrandall22 added On Staging Will be merged to the staging branch by Github Actions and removed On Staging Will be merged to the staging branch by Github Actions labels Mar 12, 2025
Copy link
Contributor

@canac canac left a comment

Choose a reason for hiding this comment

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

I could also see potential for needing to move the agreement below the checkbox.

I also expected the agreement to appear below the checkbox, but we can see what stakeholders think.

Great work on this!

@wrandall22 wrandall22 force-pushed the EP-2564-minimize-eft-agreement branch from 35445c2 to 61ae0e7 Compare March 13, 2025 19:33
@wrandall22 wrandall22 force-pushed the EP-2564-minimize-eft-agreement branch from c85137e to 32945f0 Compare March 24, 2025 14:39
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.

2 participants