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

Get address fields by dataScope instead of name #255

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

Conversation

woutk88
Copy link

@woutk88 woutk88 commented Jun 8, 2021

After updating to version 1.9.7 or higher, timeframes and locations aren't loading anymore for guest customers, because we've moved (re-ordered and re-grouped) address fields in the checkout (for multiple clients). Turns out this is happening because from version 1.9.7 the address fields are retrieved by name. These names are very specific and will change when a field is moved. The dataScope however should stay the same, therefore it would be more reliable and flexible to use this to get the fields.

…are still found when they are moved from the default posiiton.
@tig-jeffreybranderhorst
Copy link
Contributor

Hey @woutk88 ,

Thank you for submitting this, we will look at the pull request and place it on our backlog to add to the extension.

Have a great day,
Jeffrey

@igorwulff
Copy link

Any updates on this? It has been a while since this PR was send-in for review.

@hkruit
Copy link

hkruit commented Dec 15, 2021

Any news on this pull request?

@tig-jeffreybranderhorst
Copy link
Contributor

Hi,

Thank you for contacting us, I'm going to find out the status of this pull request and will come back with an update soon!

Have a great day,
Jeffrey

@tig-jeffreybranderhorst
Copy link
Contributor

Hi,

Like I promised, an update on this request. We couldn't get the pull request working on a clean environment. But it is still on our backlog to find a fix for this, we already tried some ideas but haven't found the right solution yet.

Have a great day,
Jeffrey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants