Skip to content

Conversation

@github-actions
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

@saschanaz
Copy link
Collaborator

@Bashamega Bashamega changed the title Add payerErrors Migrate payerErrors to KDL Dec 24, 2025
@Bashamega
Copy link
Contributor Author

I have updated.
Is there a way for me to quickly check if it is really there? I only check IDL files, and thought they were enough.
I don't know C++

@saschanaz
Copy link
Collaborator

That's specific to each implementation. In Gecko you may see:

  • Func="mozilla::dom::PaymentRequest::PrefEnabled", or
  • Pref="dom.payments.request.enabled"

Func is a bit more complex than Pref. In either cases you can search again with the string, and searchfox should show the most relevant thing on the top of the page. And then you would find something starting with StaticPrefs:: like this. If you click the line then it shows "Go to definition", which should send you to the StaticPrefList.yaml where you can see the status.

Pref is much easier as searching the string should send you right to the StaticPrefList file.

@Bashamega
Copy link
Contributor Author

Thank you very much

@saschanaz
Copy link
Collaborator

LGTM

@github-actions github-actions bot merged commit aef1090 into microsoft:main Dec 24, 2025
4 checks passed
@github-actions
Copy link
Contributor

Merging because @saschanaz is a code-owner of all the changes - thanks!

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