I just tested the latest version af Tea Commerce v. 2.3.1 with Umbraco 7.1.4 and I have some suggestions for improvements:
When you choose "cancel" in "Sort"-panel, the panel doesn't close.
If you reload a parent node, which doesn't have children you'll get an server error: Failed to retrieve data for child nodes undefined. E.g. if you choose "reload nodes" before you create a store first time or reload nodes for a country e.g. the default Jamaica. It seems it have two levels - I guess it's because you can group it into regions like Europe -> Denmark, Germany, .... North America -> USA, Canada....
Click on many of the parent nodes return: Request error: The URL returned a 404 (not found): views/tea-commerce-store-tree/edit.html
For image to shipping method or payment method you can choose the Media root node If you afterwards click "delete"it returns HTTP Error 403.14.
Maybe consider to style dialogs like "create" and "sort" consistency with Umbraco's dialog.
Sort dialog for Tea Commerce.
Sort dialog for Umbraco.
Create dialog for Tea Commerce.
Create dialog for Umbraco - e.g. create new stylesheet, template..
Regarding the default curreny - this is by design. A country region (like a state) dont have a different currency thatn its parent country (USA). So that is why :)
I noticed a minor issue in the "Search criteria" area in Tea Commerce 2.3.2 - it might also be an issue in previously versions. The for attribute refers to a different id and I think you just forgot to change it to a different field.
The for attribute with value inputPassword is on all fields in the search area. In the example above it should refer to body_TxtOrderNumber. If the textbox have ID="TxtOrderNumber" you could use ClientIDMode="static" or in for attribute write to ensure you get the right ID.
I also think it still is an issue that Tea Commerce 2.3.2 override the assembly Newtonsoft.Json.dll in /bin during install on e.g. Umbraco 7.1.4. and when you uninstall Tea Commerce it also removes the assembly unless you uncheck it from the uninstall. Perhaps it should be excluded from the build a just use the one in Umbraco? see more on this in this post.
yeah, it seems there is an issue with how Umbraco transform the PropertyPanel, where the label always seems to have added for="inputPassword" ... at least when using loading .aspx page in Umbraco 7.
When the page is in a view (a html page) it could be written with the use of an alias attribute:
Like Umbraco some places in the backoffice load .aspx pages (which I think will be rewritten in AngularJS at some point), these pages in .aspx for Tea Commerce will in future be rewritten in AngularJS?
Feedback on Tea Commerce for Umbraco 7
I just tested the latest version af Tea Commerce v. 2.3.1 with Umbraco 7.1.4 and I have some suggestions for improvements:
Futhermore, if it's the case you can group the countries into multiple levels then the second level is missing "Default currency" incl. in the dialog:
Jamaica:
Test:
Hi Bjarne
Thanks for your great feedback!!
Regarding the default curreny - this is by design. A country region (like a state) dont have a different currency thatn its parent country (USA). So that is why :)
Kind regards
Anders
Hi Anders
Okay, so the structure is intended to be like:
Hi Anders
I noticed a minor issue in the "Search criteria" area in Tea Commerce 2.3.2 - it might also be an issue in previously versions.
The for attribute refers to a different id and I think you just forgot to change it to a different field.
The for attribute with value inputPassword is on all fields in the search area. In the example above it should refer to body_TxtOrderNumber. If the textbox have ID="TxtOrderNumber" you could use ClientIDMode="static" or in for attribute write to ensure you get the right ID.
I also think it still is an issue that Tea Commerce 2.3.2 override the assembly Newtonsoft.Json.dll in /bin during install on e.g. Umbraco 7.1.4. and when you uninstall Tea Commerce it also removes the assembly unless you uncheck it from the uninstall. Perhaps it should be excluded from the build a just use the one in Umbraco? see more on this in this post.
Thanks.
/Bjarne
Hi Bjarne
That is Umbraco that does that. We just use their controls. So they properly has that error in the internals of Umbraco :)
Hi Anders
yeah, it seems there is an issue with how Umbraco transform the PropertyPanel, where the label always seems to have added for="inputPassword" ... at least when using loading .aspx page in Umbraco 7.
When the page is in a view (a html page) it could be written with the use of an alias attribute:
Like Umbraco some places in the backoffice load .aspx pages (which I think will be rewritten in AngularJS at some point), these pages in .aspx for Tea Commerce will in future be rewritten in AngularJS?
/Bjarne
We will make it AngularJS when the Umbraco 7 API platform for custom sections and custom UI is stable and well polished :)
Kind regards
Anders
is working on a reply...