This repository has been archived by the owner on Mar 9, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 12
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #11 from mageplaza/2.3-develop
Clean code
- Loading branch information
Showing
9 changed files
with
278 additions
and
275 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,34 +1,34 @@ | ||
| --- | ||
| name: Bug report | ||
| about: Technical issue with the extension | ||
| labels: 'Issue' | ||
|
|
||
| --- | ||
|
|
||
| <!--- | ||
| Fields marked with (*) are required. Please don't remove the template. | ||
| --> | ||
|
|
||
| ### Preconditions (*) | ||
| <!--- | ||
| Provide the exact Magento version (example: 2.4.0) and any important information on the environment where bug is reproducible. | ||
| --> | ||
| 1. | ||
| 2. | ||
|
|
||
| ### Steps to reproduce (*) | ||
| <!--- | ||
| Important: Provide a set of clear steps to reproduce this bug. We can not provide support without clear instructions on how to reproduce. | ||
| --> | ||
| 1. | ||
| 2. | ||
|
|
||
| ### Expected result (*) | ||
| <!--- Tell us what do you expect to happen. --> | ||
| 1. [Screenshots, logs or description] | ||
| 2. | ||
|
|
||
| ### Actual result (*) | ||
| <!--- Tell us what happened instead. Include error messages and issues. --> | ||
| 1. [Screenshots, logs or description] | ||
| 2. | ||
| --- | ||
| name: Bug report | ||
| about: Technical issue with the extension | ||
| labels: 'Issue' | ||
|
|
||
| --- | ||
|
|
||
| <!--- | ||
| Fields marked with (*) are required. Please don't remove the template. | ||
| --> | ||
|
|
||
| ### Preconditions (*) | ||
| <!--- | ||
| Provide the exact Magento version (example: 2.4.0) and any important information on the environment where bug is reproducible. | ||
| --> | ||
| 1. | ||
| 2. | ||
|
|
||
| ### Steps to reproduce (*) | ||
| <!--- | ||
| Important: Provide a set of clear steps to reproduce this bug. We can not provide support without clear instructions on how to reproduce. | ||
| --> | ||
| 1. | ||
| 2. | ||
|
|
||
| ### Expected result (*) | ||
| <!--- Tell us what do you expect to happen. --> | ||
| 1. [Screenshots, logs or description] | ||
| 2. | ||
|
|
||
| ### Actual result (*) | ||
| <!--- Tell us what happened instead. Include error messages and issues. --> | ||
| 1. [Screenshots, logs or description] | ||
| 2. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,19 +1,19 @@ | ||
| --- | ||
| name: Developer experience issue | ||
| about: Issues related to customization, extensibility, modularity | ||
| labels: 'Experience' | ||
|
|
||
| --- | ||
|
|
||
| <!--- | ||
| Fields marked with (*) are required. Please don't remove the template. | ||
| --> | ||
|
|
||
| ### Summary (*) | ||
| <!--- Describe the issue you are experiencing. Include general information, error messages, environments, and so on. --> | ||
|
|
||
| ### Examples (*) | ||
| <!--- Provide code examples or a patch with a test (recommended) to clearly indicate the problem. --> | ||
|
|
||
| ### Proposed solution | ||
| <!--- Suggest your potential solutions for this issue. --> | ||
| --- | ||
| name: Developer experience issue | ||
| about: Issues related to customization, extensibility, modularity | ||
| labels: 'Experience' | ||
|
|
||
| --- | ||
|
|
||
| <!--- | ||
| Fields marked with (*) are required. Please don't remove the template. | ||
| --> | ||
|
|
||
| ### Summary (*) | ||
| <!--- Describe the issue you are experiencing. Include general information, error messages, environments, and so on. --> | ||
|
|
||
| ### Examples (*) | ||
| <!--- Provide code examples or a patch with a test (recommended) to clearly indicate the problem. --> | ||
|
|
||
| ### Proposed solution | ||
| <!--- Suggest your potential solutions for this issue. --> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,22 +1,22 @@ | ||
| --- | ||
| name: Feature request | ||
| about: Request feature for development | ||
| labels: 'Feature Request' | ||
|
|
||
| --- | ||
|
|
||
| <!--- | ||
| Fields marked with (*) are required. Please don't remove the template. | ||
| --> | ||
|
|
||
| ### Description (*) | ||
| <!--- Describe the feature you would like to add. --> | ||
|
|
||
| ### Expected behavior (*) | ||
| <!--- What is the expected behavior of this feature? How is it going to work? --> | ||
|
|
||
| ### Benefits | ||
| <!--- How do you think this feature would improve the extension? --> | ||
|
|
||
| ### Additional information | ||
| <!--- What other information can you provide about the desired feature? --> | ||
| --- | ||
| name: Feature request | ||
| about: Request feature for development | ||
| labels: 'Feature Request' | ||
|
|
||
| --- | ||
|
|
||
| <!--- | ||
| Fields marked with (*) are required. Please don't remove the template. | ||
| --> | ||
|
|
||
| ### Description (*) | ||
| <!--- Describe the feature you would like to add. --> | ||
|
|
||
| ### Expected behavior (*) | ||
| <!--- What is the expected behavior of this feature? How is it going to work? --> | ||
|
|
||
| ### Benefits | ||
| <!--- How do you think this feature would improve the extension? --> | ||
|
|
||
| ### Additional information | ||
| <!--- What other information can you provide about the desired feature? --> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,19 +1,19 @@ | ||
| <!--- Provide a general summary of the Pull Request in the Title above --> | ||
|
|
||
| ### Description | ||
| <!--- Provide a description of the changes proposed in the pull request --> | ||
|
|
||
| ### Fixed Issues (if relevant) | ||
| <!--- Provide a list of fixed issues in the format magento/magento2#<issue_number>, if relevant --> | ||
| 1. https://github.com/mageplaza/magento-2-reports/<issue_number>: Issue title | ||
| 2. ... | ||
|
|
||
| ### Manual testing scenarios | ||
| <!--- Provide a set of unambiguous steps to test the proposed code change --> | ||
| 1. | ||
| 2. | ||
|
|
||
| ### Contribution checklist | ||
| - [ ] Pull request has a meaningful description of its purpose | ||
| - [ ] All commits are accompanied by meaningful commit messages | ||
| - [ ] All new or changed code is covered with unit/integration tests (if applicable) | ||
| <!--- Provide a general summary of the Pull Request in the Title above --> | ||
|
|
||
| ### Description | ||
| <!--- Provide a description of the changes proposed in the pull request --> | ||
|
|
||
| ### Fixed Issues (if relevant) | ||
| <!--- Provide a list of fixed issues in the format magento/magento2#<issue_number>, if relevant --> | ||
| 1. https://github.com/mageplaza/magento-2-reports/<issue_number>: Issue title | ||
| 2. ... | ||
|
|
||
| ### Manual testing scenarios | ||
| <!--- Provide a set of unambiguous steps to test the proposed code change --> | ||
| 1. | ||
| 2. | ||
|
|
||
| ### Contribution checklist | ||
| - [ ] Pull request has a meaningful description of its purpose | ||
| - [ ] All commits are accompanied by meaningful commit messages | ||
| - [ ] All new or changed code is covered with unit/integration tests (if applicable) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,13 @@ | ||
| # Configuration for probot-no-response - https://github.com/probot/no-response | ||
|
|
||
| # Number of days of inactivity before an Issue is closed for lack of response | ||
| daysUntilClose: 7 | ||
| # Label requiring a response | ||
| responseRequiredLabel: waiting-customer-response | ||
| # Comment to post when closing an Issue for lack of response. Set to `false` to disable | ||
| closeComment: > | ||
| This issue has been automatically closed because there has been no response | ||
| to our request for more information from the original author. With only the | ||
| information that is currently in the issue, we don't have enough information | ||
| to take action. Please reach out if you have or find the answers we need so | ||
| that we can investigate further. | ||
| # Configuration for probot-no-response - https://github.com/probot/no-response | ||
|
|
||
| # Number of days of inactivity before an Issue is closed for lack of response | ||
| daysUntilClose: 7 | ||
| # Label requiring a response | ||
| responseRequiredLabel: waiting-customer-response | ||
| # Comment to post when closing an Issue for lack of response. Set to `false` to disable | ||
| closeComment: > | ||
| This issue has been automatically closed because there has been no response | ||
| to our request for more information from the original author. With only the | ||
| information that is currently in the issue, we don't have enough information | ||
| to take action. Please reach out if you have or find the answers we need so | ||
| that we can investigate further. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,36 @@ | ||
| <?xml version="1.0"?> | ||
| <!-- | ||
| /** | ||
| * Mageplaza | ||
| * | ||
| * NOTICE OF LICENSE | ||
| * | ||
| * This source file is subject to the Mageplaza.com license that is | ||
| * available through the world-wide-web at this URL: | ||
| * https://www.mageplaza.com/LICENSE.txt | ||
| * | ||
| * DISCLAIMER | ||
| * | ||
| * Do not edit or add to this file if you wish to upgrade this extension to newer | ||
| * version in the future. | ||
| * | ||
| * @category Mageplaza | ||
| * @package Mageplaza_LoginAsCustomer | ||
| * @copyright Copyright (c) Mageplaza (https://www.mageplaza.com/) | ||
| * @license https://www.mageplaza.com/LICENSE.txt | ||
| */ | ||
| --> | ||
| <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd"> | ||
| <type name="Magento\Sales\Block\Adminhtml\Order\View"> | ||
| <plugin name="add_login_button" type="Mageplaza\LoginAsCustomer\Plugin\AddButton"/> | ||
| </type> | ||
| <type name="Magento\Ui\Model\Export\MetadataProvider"> | ||
| <arguments> | ||
| <argument name="data" xsi:type="array"> | ||
| <item name="mploginascustomer_log_listing" xsi:type="array"> | ||
| <item name="created_at" xsi:type="string">created_at</item> | ||
| </item> | ||
| </argument> | ||
| </arguments> | ||
| </type> | ||
| </config> | ||
| <?xml version="1.0"?> | ||
| <!-- | ||
| /** | ||
| * Mageplaza | ||
| * | ||
| * NOTICE OF LICENSE | ||
| * | ||
| * This source file is subject to the Mageplaza.com license that is | ||
| * available through the world-wide-web at this URL: | ||
| * https://www.mageplaza.com/LICENSE.txt | ||
| * | ||
| * DISCLAIMER | ||
| * | ||
| * Do not edit or add to this file if you wish to upgrade this extension to newer | ||
| * version in the future. | ||
| * | ||
| * @category Mageplaza | ||
| * @package Mageplaza_LoginAsCustomer | ||
| * @copyright Copyright (c) Mageplaza (https://www.mageplaza.com/) | ||
| * @license https://www.mageplaza.com/LICENSE.txt | ||
| */ | ||
| --> | ||
| <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd"> | ||
| <type name="Magento\Sales\Block\Adminhtml\Order\View"> | ||
| <plugin name="add_login_button" type="Mageplaza\LoginAsCustomer\Plugin\AddButton"/> | ||
| </type> | ||
| <type name="Magento\Ui\Model\Export\MetadataProvider"> | ||
| <arguments> | ||
| <argument name="data" xsi:type="array"> | ||
| <item name="mploginascustomer_log_listing" xsi:type="array"> | ||
| <item name="created_at" xsi:type="string">created_at</item> | ||
| </item> | ||
| </argument> | ||
| </arguments> | ||
| </type> | ||
| </config> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.