Hi Lin,
Thanks for the corrections, it now LGTM. I mark it as approved, but I just saw one little thing more here :
0 + 'depends': ['product','stock'],#TO REMOVE joomlaconnector for standard
=> You should remove the comment about "joomlconnector", that doesn't make any sense I think
Regards,
Joël
« Back to merge proposal
Hi Lin,
Thanks for the corrections, it now LGTM. I mark it as approved, but I just saw one little thing more here :
0 + 'depends': ['product' ,'stock' ],#TO REMOVE joomlaconnector for standard
=> You should remove the comment about "joomlconnector", that doesn't make any sense I think
Regards,
Joël