Hi ,

Apart from reviewing the code changes, please make sure to review the testing instructions as well.

You can follow this guide to find out what good testing instructions should look like:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

Thanks for submitting this!

Neither ellipses nor asterisks align with the way some popular services such as GitHub or Google Cloud Platform do things (not that we need to align with them) but, anecdotally, I think you're right that asterisks are a more common pattern.

Can you update the PR description, expand the details section and populate the information we need for a changelog?

Thanks for submitting this!

Neither ellipses nor asterisks align with the way some popular services such as GitHub or Google Cloud Platform do things (not that we need to align with them) but, anecdotally, I think you're right that asterisks are a more common pattern.

Can you update the PR description, expand the details section and populate the information we need for a changelog?

Thank you Barry. I have updated the details and added the changelog.

Significance: patch
Type: tweak

When displaying partial consumer keys in the REST API settings, replace ellipses with asterisks.

Unfortunately, your branch permissions won't let our workflow generate the changelog file. Can you add the above to plugins/woocommerce/changelog/fix-39194? Our automated checks will not let us merge without the changelog.

Seems reasonable, but we will need the changelog entry to merge (branch permissions mean we can't do this for you).

@barryhughes I have added the changelog, however I notice that the 'Changelog Auto Add' Check is still failing. Could you kindly help to point me in the direction to fix this? Thank you.

Thanks! We are able to ignore that specific check, so long as the other required checks pass. I'm just waiting for what I think is a flaky test to re-run and pass—then I'll go ahead and merge.