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-2525 - Hide unnecessary fields #1120

Merged
merged 8 commits into from
Dec 16, 2024

Conversation

wjames111
Copy link

@wjames111 wjames111 commented Nov 26, 2024

There is a desire to reduce friction by removing fields that are unnecessary for checkout to work properly. The new config option should hide/show all of these fields. Currently proposed fields:

  • Middle Initial
  • ‘Optional’ section - ie notes
    -Additional your information header
  • Suffix
  • Phone Number
  • Spouse information

Jira Ticket EP-2525

@wjames111 wjames111 self-assigned this Nov 26, 2024
@wjames111 wjames111 changed the base branch from EP-2519 to EP-2517-branded-checkout-improvements November 27, 2024 19:59
@wjames111 wjames111 requested a review from dr-bizz November 27, 2024 20:04
Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

These are my intially thoughts. Nothing crazy, but code styling. I will test the PR on my local.

@dr-bizz dr-bizz self-requested a review November 27, 2024 20:14
src/app/designationEditor/designationEditor.tpl.html Outdated Show resolved Hide resolved
src/app/branded/step-1/branded-checkout-step-1.tpl.html Outdated Show resolved Hide resolved
@@ -214,7 +214,7 @@ <h4 class="panel-title border-bottom-small mt0 pt-" translate>
</div><!-- // panel -->
</div>
</div>
<div class="panel panel-default give-modal-panel">
<div ng-if="$ctrl.useV3 !== 'true'" class="panel panel-default give-modal-panel">
Copy link
Contributor

Choose a reason for hiding this comment

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

One concern I realize I have with this is our datadog tests. These rely on the special instructions in order to work. We'll need to make sure that our tests are pointing at a v2 branded checkout.

src/common/components/addressForm/addressForm.tpl.html Outdated Show resolved Hide resolved
src/common/components/contactInfo/contactInfo.component.js Outdated Show resolved Hide resolved
src/common/components/addressForm/addressForm.component.js Outdated Show resolved Hide resolved
Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

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

We need to ensure the country is present on this form.

When I enter my British address, I'm unable to change the country, which means the input fields doesn't change to the International input fields. This means it requires me to select a US State, which is wrong.

69 Drayton Place, (Address)
Irthlingborough, (city)
Northamptonshire, (County)
NN9 5TD, (post code)
England (country)

src/common/components/addressForm/addressForm.tpl.html Outdated Show resolved Hide resolved
@dr-bizz
Copy link
Contributor

dr-bizz commented Dec 2, 2024

This feedback needs to go back to the stakeholders, and we need to come up with that 1. removes unnecessary fields without removing one users' needs and 2. allows the user to enter the field if they desire.

Copy link
Contributor

@wrandall22 wrandall22 left a comment

Choose a reason for hiding this comment

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

It may be worth updating the PR description to note which fields this PR is actually hiding.

@wjames111 wjames111 merged commit 41840af into EP-2517-branded-checkout-improvements Dec 16, 2024
1 check passed
@wjames111 wjames111 deleted the EP-2525 branch December 16, 2024 17: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.

3 participants