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

Add missing configuration #6770

Merged
merged 2 commits into from
Mar 6, 2020
Merged

Conversation

anandhan-ziffity
Copy link
Contributor

@anandhan-ziffity anandhan-ziffity commented Mar 3, 2020

Purpose of this pull request

Add missing configuration
This pull request (PR) adds a missing attribute to the sample code.

Affected DevDocs pages

https://devdocs.magento.com/guides/v2.2/javascript-dev-guide/javascript/requirejs.html

@devops-devdocs
Copy link
Collaborator

An admin must run tests on this PR before it can be merged.

@rogyar
Copy link
Contributor

rogyar commented Mar 3, 2020

Hi @anandhan-ziffity. Thank you for your collaboration. Could I kindly ask you to provide more information in the PR description about the purpose of the current PR next time, please? Particularly, describe the fact why do we need this change.

Thank you!

Copy link
Contributor

@rogyar rogyar left a comment

Choose a reason for hiding this comment

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

@dobooth dobooth added 2.3.x Magento 2.3 related changes Technical Updates to the code or processes that alter the technical content of the doc labels Mar 5, 2020
@dobooth dobooth changed the base branch from master to small-changes March 6, 2020 15:47
@dobooth dobooth merged commit 7330670 into magento:small-changes Mar 6, 2020
@ghost
Copy link

ghost commented Mar 6, 2020

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.3.x Magento 2.3 related changes Partner: Ziffity partners-contribution PR created by Magento partner Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants