You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In general, the same [guidelines](https://devdocs.magento.com/guides/v2.3/contributor-guide/maintainers.html) for [`magento/magento2`](https://github.com/magento/magento2) maintainers apply to devdocs maintainers. However, there are some additional guidelines specific to devdocs that will help you as a maintainer.
- Enforce the use of the issue/pull requests template
10
+
- Ask contributors to link to the code base to validate documentation updates when applicable
11
+
- Ask contributors not to contribute to v2.0 docs
12
+
- If a maintainer creates a pull request, it should be reviewed by another maintainer or DevDocs staff member
13
+
14
+
## Labels
15
+
16
+
Labels are important because they help us identify pull requests and ensure that contributors receive points and recognition. See [devdocs awards and points](https://devdocs.magento.com/guides/v2.3/contributor-guide/contributing.html#devdocs-awards-and-points).
17
+
18
+
Here is a brief summary of the most important labels:
19
+
20
+
-`New Topic`: Entirely new documents
21
+
-`Major Update`: Significant new info: new section in existing topic, etc.
22
+
-`Technical`: Changes to technical content/code/processes/naming conventions (any change to technical content)
23
+
-`Editorial`: Typos, grammatical inconsistencies, or minor rewrites
24
+
-`small changes`: See [Small changes workflow](#small-changes-workflow)
25
+
26
+
We also use version labels when appropriate.
27
+
28
+
As a maintainer, we expect you to add or remove labels when appropriate.
29
+
30
+
See https://github.com/magento/devdocs/labels for all labels and their descriptions.
31
+
32
+
## Testing
33
+
34
+
Currently, we only test internal links. In the future, we plan to expand tests to include external links, markdown linting, and spell checking.
35
+
36
+
We use private CI/CD stack and do not provide access to it.
37
+
38
+
Every pull request to the `master` branch must pass tests before it can be merged. When a pull request is ready for tests, a member of the [`devdocs-admins`](https://github.com/orgs/magento/teams/devdocs-admins) team must add the test trigger phrase to the pull request as a comment. The trigger phrase is "_running tests_". By approving a pull request, you are signalling an admin that the pull request is ready for tests.
39
+
40
+
## Projects
41
+
42
+
We use several projects to help organize issues and pull requests. You can add existing issues and pull requests to these projects as you see fit.
43
+
44
+
https://github.com/magento/devdocs/projects
45
+
46
+
## Style
47
+
48
+
We prefer Markdown over HTML (in most cases). You can use [kramdown](https://kramdown.gettalong.org/syntax.html) syntax for more markup features and [Liquid](https://jekyllrb.com/docs/liquid/) for in-topic scripting.
49
+
50
+
## Small changes workflow
51
+
52
+
Before merging a pull request to the `master` branch, it must pass automated testing. Testing takes about 30 minutes to complete for each pull request, so we created a workflow to save time for small changes.
53
+
54
+
-**Individual pull requests to `master`**—15 individual pull requests to `master` x 30 minutes per pull request = 7.5 hrs of testing time
55
+
-**Multiple pull requests using `small_changes`**—1 `small_changes` pull request (containing 15 individual pull requests) to `master` x 30 minutes = 30 minutes of testing time
56
+
57
+
This workflow is for typos, formatting issues, and minor text additions or deletions. It is not for substantial new content, changes to tables, new files, or files that have been moved.
58
+
59
+
Periodically, we will create a pull request from `small_changes` to `master` and then run tests on that pull request to save time.
60
+
61
+
### Process
62
+
63
+
1. Review the pull request and either approve it or request changes.
64
+
1. Apply the `Small changes` label if one of the following labels should also be applied:
65
+
66
+
-`Editorial`
67
+
-`Technical`
68
+
69
+
1. That's it! A devdocs-admin will run tests and merge.
- description: Added examples of [creating and updating a Magento admin user](https://devdocs.magento.com/guides/v2.3/install-gde/install/cli/install-cli-subcommands-admin.html)
- description: Added a section and code example about using [email templates](https://devdocs.magento.com/guides/v2.3/frontend-dev-guide/translations/translate_theory.html)
- description: Added the observer interface implementation to the [observer example](https://devdocs.magento.com/guides/v2.1/coding-standards/technical-guidelines.html)
Copy file name to clipboardExpand all lines: guides/v2.1/contributor-guide/maintainers.md
+4Lines changed: 4 additions & 0 deletions
Original file line number
Diff line number
Diff line change
@@ -30,3 +30,7 @@ Maintainers take part in these main activities:
30
30
- Participate in discussions regarding proposed solutions, created issues, and more
31
31
- Help other community members resolve their questions and issues with advice, helpful code snippets, or other examples
32
32
- Find and propose better or new methods for pull request reviews, code delivery, QA activities, tests, and more
33
+
34
+
## Guidelines for DevDocs
35
+
36
+
In general, the same [guidelines](https://devdocs.magento.com/guides/v2.3/contributor-guide/maintainers.html) for [`magento/magento2`](https://github.com/magento/magento2) maintainers apply to DevDocs maintainers. However, there are some [additional guidelines](https://github.com/magento/devdocs/blob/master/.github/MAINTAINER_GUIDELINES.md) that apply to DevDocs.
Use the [`ActionFactory`] in your router to create an instance of an `Action` class.
162
162
163
+
164
+
{: .bs-callout .bs-callout-info }
165
+
Action class should return a `result object`.
166
+
167
+
## Result object
168
+
169
+
Name | Description
170
+
--- | ---
171
+
`json` | Sets `Content-Type:application/json` in the header and returns a json encoded representation of an array with data
172
+
`raw` | Returns the data as it's been set. Does not set a `Content-Type` in the header
173
+
`redirect` | Creates an external redirect, which the browser follows and requests a new url
174
+
`forward` | Internally calls the execute method of another action class and does not trigger a new request from the browser. The URL stays the same
175
+
`layout` | View result. You can use a generic layout response to render any kind of layout. The layout comprises a response body from its layout elements and sets it to the HTTP response
176
+
`page` | View result. Encapsulates page type, page configuration, and imposes certain layout handles. `page` triggers `layout.xml` to render into HTML
0 commit comments