Skip to content
This repository has been archived by the owner on Dec 19, 2019. It is now read-only.

url_path is not working as documented #713

Closed
TomashKhamlai opened this issue May 16, 2019 · 1 comment
Closed

url_path is not working as documented #713

TomashKhamlai opened this issue May 16, 2019 · 1 comment

Comments

@TomashKhamlai
Copy link
Contributor

TomashKhamlai commented May 16, 2019

Preconditions (*)

  1. Simple Product is created and assigned to the category "sale"

Steps to reproduce (*)

  1. Perform query
{
  products(
    search: "simple"
    filter: {
      sku: {
        eq: "simple"
      }
    }
  ) {
    items {
      type_id
      sku
      name
      url_key
      url_path
      canonical_url
      url_rewrites {
        url
        url
      }
    }
    total_count
  }
}

Expected result (*)

  1. Works according to documentation
Attribute Data Type Description
url_path String The part of the URL that precedes the url_key.
This attribute is defined in the CatalogUrlRewrite module

Actual result (*)

  1. Response:
{
  "data": {
    "products": {
      "items": [
        {
          "type_id": "simple",
          "sku": "simple",
          "name": "simple",
          "url_key": "simple",
          "url_path": null,
          "canonical_url": "http://graph-sd.io/simple.html",
          "url_rewrites": [
            {
              "url": "simple.html"
            },
            {
              "url": "sale/simple.html"
            }
          ]
        }
      ],
      "total_count": 1
    }
  }
}
@TomashKhamlai TomashKhamlai assigned keharper and unassigned keharper May 16, 2019
@XxXgeoXxX XxXgeoXxX self-assigned this May 20, 2019
lenaorobei added a commit to XxXgeoXxX/graphql-ce that referenced this issue Jun 13, 2019
- added `@deprecated` tag to the `url_path`
- details magento/magento2#9113
@naydav naydav added this to the Release: 2.3.3 milestone Jun 19, 2019
@naydav
Copy link
Contributor

naydav commented Jun 19, 2019

#721

@naydav naydav closed this as completed Jun 19, 2019
magento-engcom-team pushed a commit that referenced this issue Nov 4, 2019
Added test case Id and indentation fix
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants