Matthew Wells
Matthew Wells
@cuongdv did you find a solution for this? It appears that category import explicitly ignores the nested array feature. (edit: [reference](https://github.com/avstudnitz/AvS_FastSimpleImport/blob/04979b81f74b2fec242ec5bb05e204c733257112/src/app/code/community/AvS/FastSimpleImport/Model/Import.php#L279))
> Everything works with symlink strategy (used in local environments), but problem happens when switching to copy. Same thing happened to me (version constraint: `~2.1`). The solution was to delete...
Thanks for the review, @joni-jones. I'll make that change. A couple questions for you: ## Regarding Interfaces Should I assume there needs to be a Magento-namespaced interface? For example: ```php...
@gabrieldagama, thanks for the review. I appreciate it! Regarding performance: 1. The biggest improvement will be noticed when running a common command rather than the two which cause the biggest...
@gabrieldagama regarding: > Regards the other using CommandLoader for the other commands, it may be backward incompatible, currently, developers rely on CommandListInterface to add new commands, so any change there...
> I guess we can create a separate issue for it and work this out on separate PR (once this gets merged), what do you think? Good plan Regarding static...
@magento run all tests
Hey, sorry about the delay. I got busy with other work. I fixed the errors found by [`Magento\Test\Php\LiveCodeTest`](https://public-results-storage-prod.magento-testing-service.engineering/reports/magento/magento2/pull/29355/bc59967a3e931f8357ab0caca45634e5/Statics/allure-report-b2b/index.html#suites/51d3ab4d5d77a0bc6a7ea344e9e7e6be), aside from those in `*Command` classes (which got caught up in the...
@engcom-Echo, please note my comment from a couple weeks ago.
@ihor-sviziev, thanks for giving this some attention! I'll take a look at your feedback in the next few days and change the target to 2.5-dev.