NopRoot Bootstrap theme (Free)

SilhouetteBS wrote:

Hi, Thanks a lot for your feedback. This issue is solved. Get update from here - https://noproot.codeplex.com/SourceControl.

Product specifications do not display properly in the Specification tab on the Product Details page. It looks like this: ColorYellow. It should be Color: Yellow

SilhouetteBS wrote:

Hi,
Thanks for your feedback. This issue is solved. Get latest update from here - https://noproot.codeplex.com/SourceControl/latest#README.md

Hi, awesome theme!
However, I found one bug. Rich Snippets (Structured Data) requires a product name, which is missing from at least "ProductTemplate.Simple.cshtml".
I've changed
<h3>@Model.Name</h3>
to
<h3 itemprop="name">@Model.Name</h3>

and moved

itemscope itemtype="http://schema.org/Product" data-productid="@Model.Id"

to

the most outer div with class "page product-details-page panel panel-default form-panel"

Verified with Google Structured Data testing tool that it's working.

Was trying to see if this issue was mentioned, but with nopRoot enabled, when I pick a Country on the shopping cart, the States dropdown does not get populated and I get the following error in the system Log:

The parameters dictionary contains a null entry for parameter 'addSelectStateItem' of non-nullable type 'System.Boolean' for method 'System.Web.Mvc.ActionResult GetStatesByCountryId(System.String, Boolean)' in 'Nop.Web.Controllers.CountryController'. An optional parameter must be a reference type, a nullable type, or be declared as an optional parameter. Parameter name: parameters

Nobody?  Am I the only one having an error with the State dropdown not populating ONLY when using nopRoot?

Need help please...

mbiefeni wrote:

Hi, Sorry for the late reply. We will check this issue today.

mbiefeni wrote:

Hi, Which version of nopcommerce are you using. We have just checked the version for 3.5 and it is working fine. You can download the latest version from this link ==>> https://noproot.codeplex.com/SourceControl/latest

Thanks for the reply...

I am using the latest nopCommerce 3.5.  

When I set to use the default template, no issues, but as soon as I set to nopRoot it seems to cause this issue.

You can check it out at fishingmilitia.com.  Add something to cart and check it out.  If you Inspect using Chrome you can also see that some java seems to have issues, perhaps there is a conflict with some module?  Not sure...

mbiefeni wrote:

The estimate shipping view has an incorrect parameter (addEmptyStateIfRequired)

Views\ShoppingCart\EstimateShipping.cshtml

-                 data: { "countryId": selectedItem, "addEmptyStateIfRequired": "true" },
+                 data: { "countryId": selectedItem, "addSelectStateItem": "false" },

This change was made in http://nopcommerce.codeplex.com/SourceControl/changeset/40a4c17e5ce7ca854e4a03e1b72c2923e89c0bb0, so other views may also need to be changed.

---

You can run the following in the Firebug console on the /cart page to temporarily (change doesn't persist on page reload) see the change work:

$("#CountryId").off('change');
$("#CountryId").change(function () {
              var selectedItem = $(this).val();
              var ddlStates = $("#StateProvinceId");
              var estimateProgress = $("#estimate-shipping-loading-progress");
              estimateProgress.show();
              $.ajax({
                    cache: false,
                    type: "GET",
                    url: "/country/getstatesbycountryid",
                    data: { "countryId": selectedItem, "addSelectStateItem": "false" },
                    success: function (data) {
                       ddlStates.html('');
                       $.each(data, function (id, option) {
                           ddlStates.append($('<option></option>').val(option.id).html(option.name));
                       });
                       estimateProgress.hide();
                    },
                    error: function (xhr, ajaxOptions, thrownError) {
                       alert('Failed to retrieve states.');
                       estimateProgress.hide();
                    }
                  });
          });