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

Small changes #4872

Merged
merged 84 commits into from
Jun 28, 2019
Merged

Small changes #4872

merged 84 commits into from
Jun 28, 2019

Conversation

Jakob Meissner and others added 30 commits May 2, 2019 12:41
Custom step is hidden when using back button
As mentioned under Minor it can include breaking changes, which as per SemVer should make it a Major release.
For Alert widget, Add __('<your_string>') method in Example because this method is used for translation of the string.
Add __('<your_string>') method in Alert Widget
Fix description of theme.xml contents
Fixed up formatting errors in 'Applying patches' topic.
The Magento website (https://devdocs.magento.com/guides/v2.3/config-guide/cli/config-cli-subcommands-mode.html) renders the <magento_root> tags as html, so they are not visible to the reader.
Fix root to be visible on Magento DevDocs website
Add __('<your_string>') method in Confirm Widget Example
Update docs about composer-packaged themes to include naming convention
Add content related to loader widget
Add __('<your_string>') method in Confirm Widget Examples
In line with magento2/architecture#170
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jun 28, 2019

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
9 out of 10 committers have signed the CLA.

✅ fooman
✅ hardikm-cirkleinc
✅ dobooth
✅ DrewML
✅ navarr
✅ eduard13
✅ jeff-matthews
✅ ParthBrahmbhatt
✅ twocs
❌ jaykobi

@dobooth dobooth self-assigned this Jun 28, 2019
@m2-community-project
Copy link

@dobooth unfortunately, only members of the maintainers team are allowed to assign developers to the pull request

@bdenham bdenham self-requested a review June 28, 2019 19:21
Copy link
Contributor

@bdenham bdenham left a comment

Choose a reason for hiding this comment

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

Looks good. Just one minor serial comma missing.

@dobooth
Copy link
Contributor Author

dobooth commented Jun 28, 2019

running tests

@dobooth dobooth merged commit 098bccd into master Jun 28, 2019
@ghost
Copy link

ghost commented Jun 28, 2019

Hi @dobooth, 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.

@dobooth dobooth deleted the small_changes branch June 28, 2019 19:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.