Jarvis

Results 128 comments of Jarvis

Hi, thanks for this contribution! I found some issues with the Pull Request description: * The `How to test` shouldn't be empty. ([Read explanation](https://devdocs.prestashop-project.org/9/contribute/contribution-guidelines/pull-requests/#how-to-test)) Would you mind having a look...

Hi, thanks for this contribution! I found some issues with the Pull Request description: * The `How to test` shouldn't be empty. ([Read explanation](https://devdocs.prestashop-project.org/9/contribute/contribution-guidelines/pull-requests/#how-to-test)) Would you mind having a look...

This pull request seems to contain new translation strings. I have summarized them below to ease up review: - `Admin.Advparameters.Feature` - [ ] `Select your default file separator` (Note: this...

Hi, thanks for this contribution! I found some issues with the Pull Request description: * The `deprecations` should be `yes` or `no`. ([Read explanation](https://devdocs.prestashop-project.org/9/contribute/contribution-guidelines/pull-requests/#deprecations)) Would you mind having a look...

QA approved, well done! Message to the maintainers: do not forget to milestone it before the merge.

Hi, thanks for this contribution! I found some issues with the Pull Request description: * The `branch` should be `develop` or `8.1.x`. ([Read explanation](https://devdocs.prestashop-project.org/9/contribute/contribution-guidelines/pull-requests/#branch)) * The `How to test` shouldn't...

Hi, thanks for this contribution! I found some issues with the Pull Request description: * The `How to test` shouldn't be empty. ([Read explanation](https://devdocs.prestashop-project.org/9/contribute/contribution-guidelines/pull-requests/#how-to-test)) Would you mind having a look...

QA approved, well done! Message to the maintainers: do not forget to milestone it before the merge.

This pull request seems to contain new translation strings. I have summarized them below to ease up review: - `Admin.Notifications.Error` - [ ] `Test property does not exist in the...

Hi, thanks for this contribution! I found some issues with the Pull Request description: * The `description` shouldn't be empty. ([Read explanation](https://devdocs.prestashop-project.org/9/contribute/contribution-guidelines/pull-requests/#description)) * The `type` should be one of these:...