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

Add regionCode attribute to PaymentAddress (1.1) #836

Closed
wants to merge 6 commits into from

Conversation

marcoscaceres
Copy link
Member

@marcoscaceres marcoscaceres commented Feb 21, 2019

Revert "Remove regionCode attribute (#823)"
This reverts commit 5906b72.

closes w3c/contact-picker#65

The following tasks have been completed:

  • Confirmed there are no ReSpec errors/warnings.
  • Modified Web platform tests
  • Modified MDN Docs (link)
  • Has undergone security/privacy review (link)

Implementation commitment:

  • Safari
  • Chrome
  • Firefox - partial implementation - supports the attribute.
  • Edge (public signal)

Optional, impact on Payment Handler spec?


Preview | Diff


Preview | Diff

@rsolomakhin
Copy link
Collaborator

Is this for 1.0 or 1.1+?

@marcoscaceres
Copy link
Member Author

1.1

@marcoscaceres
Copy link
Member Author

Blocked on release of 1.0.

@marcoscaceres
Copy link
Member Author

Ok, tests and spec are resurrected for when when want this...

@marcoscaceres
Copy link
Member Author

@danyao or @rsolomakhin, if you have a bug on the Chromium side, could you please add it to the top of the bug?

@rsolomakhin
Copy link
Collaborator

Linked the Chrome bug.

Copy link
Collaborator

@rsolomakhin rsolomakhin left a comment

Choose a reason for hiding this comment

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

For when you need to merge it in the future.

@marcoscaceres
Copy link
Member Author

Mucho appreciated!

Marcos Cáceres added 2 commits April 10, 2019 12:14
Revert "Remove regionCode attribute (#823)"
This reverts commit 5906b72.
@marcoscaceres
Copy link
Member Author

Similarly here, we need commitment from another implementer to land this.

@marcoscaceres marcoscaceres changed the title Add regionCode attribute to PaymentAddress Add regionCode attribute to PaymentAddress (1.1) Sep 21, 2020
@marcoscaceres
Copy link
Member Author

No moar PaymentAddress with #955

@marcoscaceres marcoscaceres deleted the regioncode_redux branch July 8, 2024 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ContactAddress 'region' inconsistencies
2 participants