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

fix: save company name(s) at address after b2b customer registration #1444

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

SGrueber
Copy link
Collaborator

@SGrueber SGrueber commented Jun 5, 2023

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

Company name is not stored on address during registration of a b2b customer.

Issue Number: Closes #

What Is the New Behavior?

Company name is saved at the address after registration.

Steps To Repeat

B2B PWA:

  • Click on registration
  • Select business customer
  • Fill in all required fields plus company name2, tax number, VAT number
  • My Account
    Profile settings > Company profile =Company name plus company name2 ✅
    Addresses > Address from registration = Company name plus company name2 is missing
    --> expected: Company name 1+2 are stored at the address

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

AB#86558

@SGrueber SGrueber self-assigned this Jun 5, 2023
@SGrueber SGrueber added the bug Something isn't working label Jun 5, 2023
@SGrueber SGrueber added this to the 4.1 milestone Jun 5, 2023
@SGrueber SGrueber merged commit 4d81e6b into develop Jun 5, 2023
@SGrueber SGrueber deleted the bugfix/companyName_after_registration branch June 5, 2023 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants