Skip to content

Remove PaymentAddress' languageCode #765

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

Merged
merged 1 commit into from
Oct 18, 2018
Merged

Conversation

marcoscaceres
Copy link
Member

@marcoscaceres marcoscaceres commented Aug 27, 2018

closes #608

The following tasks have been completed:

Implementation commitment:

Impact on Payment Handler spec?
None


Preview | Diff

marcoscaceres added a commit to web-platform-tests/wpt that referenced this pull request Aug 27, 2018
marcoscaceres added a commit to web-platform-tests/wpt that referenced this pull request Aug 27, 2018
@marcoscaceres
Copy link
Member Author

Updated MDN, and sent a PR to WTP to remove languageCode.

@marcoscaceres
Copy link
Member Author

Chairs, we should decide on a timeframe for feedback on this from the community.

1 month CFC?

I'm going to remove from Gecko in the mean time. Can always revert it later if we need it.

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.

I'm adding a usage counter for languageCode in Chrome and will get results from stable in ~2 months.

@rsolomakhin
Copy link
Collaborator

The usage counter will inform our decision to remove the languageCode immediately or after some developer outreach.

@marcoscaceres
Copy link
Member Author

@rsolomakhin

The usage counter will inform our decision to remove the languageCode immediately or after some developer outreach.

Great! Will you be notified automatically once the results are collected? I'll wait for your results before merging.

@rsolomakhin
Copy link
Collaborator

Will you be notified automatically once the results are collected?

I wish! I have a calendar reminder to check the data :-)

@marcoscaceres
Copy link
Member Author

I wish! I have a calendar reminder to check the data :-)

Good enough 👍

@marcoscaceres
Copy link
Member Author

Removed from Firefox.

@aestes
Copy link
Collaborator

aestes commented Sep 4, 2018

I've removed it from Safari as well.

@adrianhopebailie
Copy link
Collaborator

@marcoscaceres please tag me or @mountainhippo if you need anything from the chairs. Only saw this now.

@ianbjacobs is this covered by process? We never marked this as at risk previously

@ianbjacobs
Copy link
Collaborator

Hi all,

I had hoped we would hold off merging until after the CfC. That is why the draft CfC I circulated pointed to the pull request. My expectation was to merge after the CfC.

Ian

@marcoscaceres
Copy link
Member Author

Yep, that’s the plan. It’s actually to wait for @rsolomakhin to gather data from Chrome before we make a final decision.

@marcoscaceres
Copy link
Member Author

@marcoscaceres
Copy link
Member Author

@rsolomakhin, how’s that data looking? Any chance we will know by TPAC? We’ve removed this from Firefox and it’s also gone from Safari. Hopefully it’s all good news on the Chrome side.

@rsolomakhin
Copy link
Collaborator

Data shows we can remove from Chrome. I've sent out the intent to deprecate.

@rsolomakhin
Copy link
Collaborator

So, we will print a deprecation warning in developer console for a couple of months, then will remove it outright.

@marcoscaceres
Copy link
Member Author

Awesome, thanks for confirming and doing the due diligence! Going to merge this then.

@marcoscaceres marcoscaceres merged commit 3fb7e37 into gh-pages Oct 18, 2018
@marcoscaceres marcoscaceres deleted the remove_language_cod branch October 18, 2018 14:32
sideshowbarker added a commit to w3c/browser-compat-data that referenced this pull request Jan 29, 2020
queengooborg pushed a commit to mdn/browser-compat-data that referenced this pull request Jan 30, 2020
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.

Describe how to implement PaymentAddress.languageCode
5 participants