Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Weekly digest #4620

Merged
merged 3 commits into from
May 28, 2019
Merged

Weekly digest #4620

merged 3 commits into from
May 28, 2019

Conversation

dshevtsov
Copy link
Collaborator

Purpose of this pull request

This pull request is a weekly digest update.

Affected DevDocs pages

@dshevtsov dshevtsov added Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies 2.x labels May 28, 2019
@dshevtsov dshevtsov requested review from shrielenee and lorikrell May 28, 2019 15:41
@dshevtsov dshevtsov self-assigned this May 28, 2019
Copy link

@lorikrell lorikrell left a comment

Choose a reason for hiding this comment

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

Just a couple tiny edits.

type: Major update
date: May 23, 2019
link: https://github.com/magento/devdocs/pull/4590
- description: 'Reformatted YAML samples in the [Configure Environments: Services

Choose a reason for hiding this comment

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

Is this a random single quote around this sentence ' ', also period?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The quote is added by the YAML generator. It is added when the string contains special symbols. YAML or :could be special in this case I guess.
The period has to be added to the original PR #4574.

date: May 22, 2019
link: https://github.com/magento/devdocs/pull/4574
- description: Added a [new tutorial](https://devdocs.magento.com/guides/v2.3/ext-best-practices/tutorials/dynamic-row-system-config.html)
that shows you how to add a dynamic rows system configuration

Choose a reason for hiding this comment

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

Just a period

Suggested change
that shows you how to add a dynamic rows system configuration
that shows you how to add a dynamic rows system configuration.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The period has to be added into whatsnew at the original PR #4528.

@dshevtsov
Copy link
Collaborator Author

running tests

@dshevtsov dshevtsov merged commit 401cf9e into master May 28, 2019
@ghost
Copy link

ghost commented May 28, 2019

Hi @dshevtsov, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@dshevtsov dshevtsov deleted the ds_whatsnew branch May 28, 2019 21:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants