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

Maps Block: Fix typos in copy of API endpoint #11161

Merged
merged 1 commit into from
Jan 23, 2019
Merged

Maps Block: Fix typos in copy of API endpoint #11161

merged 1 commit into from
Jan 23, 2019

Conversation

akirk
Copy link
Member

@akirk akirk commented Jan 18, 2019

Our translators have discovered some typos in the descriptions introduced in #10742.

props @iqatrophie

@akirk akirk requested review from enejb and a team January 18, 2019 16:08
@matticbot
Copy link
Contributor

D23316-code. (newly created revision)

@@ -78,15 +78,15 @@ public function get_public_item_schema() {
'type' => 'object',
'properties' => array(
'code' => array(
'description' => __( 'Displays success if the oppration was successfully executed and an error code if it was not', 'jetpack' ),
'description' => __( 'Displays success if the operation was successfully executed and an error code if it was not', 'jetpack' ),
Copy link
Member Author

@akirk akirk Jan 18, 2019

Choose a reason for hiding this comment

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

@enejb Should success be in quotes since this will be the actual text of code?

@jetpackbot
Copy link

Warnings
⚠️

The PR is missing at least one [Status] label. Suggestions: [Status] In Progress, [Status] Needs Review

⚠️ "Testing instructions" are missing for this PR. Please add some
⚠️ "Proposed changelog entry" is missing for this PR. Please include any meaningful changes

This is automated check which relies on PULL_REQUEST_TEMPLATE.We encourage you to follow that template as it helps Jetpack maintainers do their job. If you think 'Testing instructions' or 'Proposed changelog entry' are not needed for your PR - please explain why you think so. Thanks for cooperation 🤖

Generated by 🚫 dangerJS against 99d9c17

@jeherve jeherve added [Block] Map [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. [Feature] WP REST API labels Jan 18, 2019
@jeherve jeherve added this to the 7.0 milestone Jan 18, 2019
@enejb
Copy link
Member

enejb commented Jan 18, 2019

Thanks for the pull request! It looks good to me!

@enejb
Copy link
Member

enejb commented Jan 18, 2019

@Automattic/jetpack-crew it looks like the code was already submitted to .com

@jeherve jeherve removed the [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. label Jan 23, 2019
Copy link
Member

@jeherve jeherve left a comment

Choose a reason for hiding this comment

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

This looks good to me. Merging.

@jeherve jeherve merged commit 75d9618 into master Jan 23, 2019
@jeherve jeherve deleted the copy-typos branch January 23, 2019 15:06
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.

5 participants