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

Wordline (Global Collect): Add agent numeric code and house number field #4847

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

yunnydang
Copy link
Contributor

@yunnydang yunnydang commented Aug 2, 2023

Local:
5563 tests, 77664 assertions, 1 failures, 19 errors, 0 pendings, 0 omissions, 0 notifications
99.6405% passed

Unit:
44 tests, 226 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
51 tests, 133 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

@yunnydang yunnydang requested a review from a team August 2, 2023 00:20
@yunnydang yunnydang self-assigned this Aug 2, 2023
Copy link
Contributor

@jcreiff jcreiff left a comment

Choose a reason for hiding this comment

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

@yunnydang I think this looks good -- while we are correcting this, can you also make the same housenumber/street correction for the shipping address logic too?

@yunnydang yunnydang force-pushed the add_housenumber_and_agent_numberic_code_fields branch from abe2e31 to fd5241e Compare August 2, 2023 20:52
Copy link
Contributor

@jcreiff jcreiff left a comment

Choose a reason for hiding this comment

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

LGTM!

@yunnydang yunnydang force-pushed the add_housenumber_and_agent_numberic_code_fields branch from fd5241e to 9fa4dce Compare August 8, 2023 19:27
@yunnydang yunnydang merged commit 9fa4dce into master Aug 8, 2023
5 checks passed
@yunnydang yunnydang deleted the add_housenumber_and_agent_numberic_code_fields branch August 8, 2023 20:04
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