Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Trunk, Release Branch 15.12
    • Fix Version/s: 16.11.01
    • Component/s: ecommerce
    • Labels:
      None
    • Sprint:
      Community Day 1 - 2016
    1. OFBIZ-6811-ecommerce-template.patch
      986 kB
      Pierre Smits
    2. OFBIZ-6811-ecommerce.patch
      1.22 MB
      Pierre Smits
    3. OFBIZ-6811.patch.bz2
      122 kB
      Nicolas Malin

      Activity

      Hide
      pfm.smits Pierre Smits added a comment -

      This patch relocates .ftl and .groovy files and addresses the issue.

      Show
      pfm.smits Pierre Smits added a comment - This patch relocates .ftl and .groovy files and addresses the issue.
      Hide
      soledad Nicolas Malin added a comment -

      Ok pierre I updated the patch with lastest community decision.

      I didn't commit because at this time I can create an order on ecommerce, with and without this. So difficult to validate it before

      Show
      soledad Nicolas Malin added a comment - Ok pierre I updated the patch with lastest community decision. I didn't commit because at this time I can create an order on ecommerce, with and without this. So difficult to validate it before
      Hide
      pfm.smits Pierre Smits added a comment -

      Thanks for submitting the revision on the patch file provided.

      I will recreate the freemarker template set into a new patch file, so that the folder name is singular (template) in stead of plural (templates). That way there is consistency in folder naming.

      Show
      pfm.smits Pierre Smits added a comment - Thanks for submitting the revision on the patch file provided. I will recreate the freemarker template set into a new patch file, so that the folder name is singular (template) in stead of plural (templates). That way there is consistency in folder naming.
      Hide
      pfm.smits Pierre Smits added a comment -

      This patch is a separation of the templates in previous patches.

      Show
      pfm.smits Pierre Smits added a comment - This patch is a separation of the templates in previous patches.
      Hide
      pfm.smits Pierre Smits added a comment -

      HI Nicolas Malin,

      I will reassign this issue to myself to provide a better patch file, as per our resolution how to handle this. See my latest comment in OFBIZ-6810

      Show
      pfm.smits Pierre Smits added a comment - HI Nicolas Malin , I will reassign this issue to myself to provide a better patch file, as per our resolution how to handle this. See my latest comment in OFBIZ-6810
      Hide
      soledad Nicolas Malin added a comment -

      don't worries I wait your patch

      Show
      soledad Nicolas Malin added a comment - don't worries I wait your patch
      Hide
      deepak.dixit Deepak Dixit added a comment -

      This has been done at r#1740485

      Thanks Pierre for your contribution.

      Show
      deepak.dixit Deepak Dixit added a comment - This has been done at r#1740485 Thanks Pierre for your contribution.
      Hide
      pfm.smits Pierre Smits added a comment -

      HI Deepak Dixit,

      Thanks for taking care of the plural 2 singular in the location (template) naming in various widgets. I would have fixed that this coming weekend as I hadn't been able to do it last days.

      Show
      pfm.smits Pierre Smits added a comment - HI Deepak Dixit , Thanks for taking care of the plural 2 singular in the location (template) naming in various widgets. I would have fixed that this coming weekend as I hadn't been able to do it last days.

        People

        • Assignee:
          deepak.dixit Deepak Dixit
          Reporter:
          pfm.smits Pierre Smits
        • Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development

              Agile