Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

GraphQL request and response correction #3652

Merged
merged 5 commits into from
Mar 3, 2019
Merged

Conversation

hiren2615
Copy link
Contributor

@hiren2615 hiren2615 commented Jan 30, 2019

Instead of "relative_url" it should be "canonical_url"

This PR is a:

  • New topic
  • Content update
  • Content fix or rewrite
  • Bug fix or improvement

Summary

When this pull request is merged, it will correct the Example usage of urlResolver endpoint

Additional information

List all affected URLs
https://devdocs.magento.com/guides/v2.3/graphql/reference/url-resolver.html

Instead of "relative_url" it should be "canonical_url"
@magento-engcom-team magento-engcom-team added Partner: Wagento PR created by Wagento partner partners-contribution PR created by Magento partner labels Jan 30, 2019
@magento-cicd2
Copy link
Contributor

An admin must run tests on this PR before it can be merged.

@jeff-matthews jeff-matthews requested a review from nuzil January 30, 2019 14:02
@jeff-matthews jeff-matthews added 2.3.x Magento 2.3 related changes Technical Updates to the code or processes that alter the technical content of the doc and removed Partner: Wagento PR created by Wagento partner labels Jan 30, 2019
@keharper
Copy link
Contributor

The attribute was renamed from canonical_url to relative_url in magento/graphql-ce#185 I'm guessing that the PR wasn't merged as quickly as I expected.

@nuzil
Copy link
Contributor

nuzil commented Jan 31, 2019

Yes correct. Basically for 2.3 it makes sense, but from 2.3.1 it will be switched back

@keharper keharper self-assigned this Feb 1, 2019
@keharper
Copy link
Contributor

keharper commented Feb 5, 2019

The rename of renamed from canonical_url to relative_url won't be merged until 2.3.2 at best. Accepting this PR.

@keharper keharper self-requested a review February 5, 2019 20:15
Copy link
Contributor

@keharper keharper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I should say I'll accept this PR with additional changes.

In the table, lines 17 and 19 relative_url must also be changed to canonical_url

@keharper
Copy link
Contributor

keharper commented Mar 3, 2019

running tests

@keharper keharper merged commit 1404fb8 into magento:master Mar 3, 2019
@ghost
Copy link

ghost commented Mar 3, 2019

Hi @hiren-wagento, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@keharper
Copy link
Contributor

keharper commented Mar 3, 2019

I finished this because it was holding up changes to be made in the 2.3.1 integration branch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.3.x Magento 2.3 related changes partners-contribution PR created by Magento partner Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants