-
Notifications
You must be signed in to change notification settings - Fork 132
Data entity document improvement #693
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
Data entity document improvement #693
Conversation
docs/data.md
Outdated
@@ -32,6 +48,12 @@ In this example: | |||
* `MAGENTO_ADMIN_USERNAME` is a name of an environment variable. | |||
The corresponding value will be assigned to `userInput` as a result. | |||
|
|||
The following is the usage of `_ENV` in the `Magento/Braintree/Test/Mftf/ActionGroup/AdminDeleteRoleActionGroup.xml` action group: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"The following is an example of the usage of..."
Same with L72 and L96.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dobooth Updated the changes.
docs/data.md
Outdated
@@ -107,7 +145,7 @@ userInput="We'll email you an order confirmation with details and tracking info. | |||
|
|||
## Format | |||
|
|||
The format of `<data>` is: | |||
The format of `<data>` entity is: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"The format of the data
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dobooth Updated the changes.
docs/data.md
Outdated
@@ -205,7 +243,7 @@ You can also call data from the xml definition of a `data` tag directly: | |||
|
|||
Attributes|Type|Use|Description | |||
---|---|---|--- | |||
`name`|string|optional|Name of the `<entity>`. | |||
`name`|string|optional|Name of the `<entity>`. Camel case is used for the entity name. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Use camel case for entity names."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dobooth Updated the changes. Please review it.
docs/data.md
Outdated
@@ -231,6 +275,12 @@ Attributes|Type|Use|Description | |||
`entityKey`|string|optional|Key attribute of the referenced entity from which to get a value. | |||
`unique`|--|--|*This attribute hasn't been implemented yet.* | |||
|
|||
Example: | |||
|
|||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use xml as the codeblock type.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dobooth This has been done. Please review it.
Description
Improved the Data entity MFTF document by providing examples for each data entity node.
Added example from the already defined MFTF test for each sub-topic.
Affected Pages
Contribution checklist