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

Conversation

mageprince
Copy link
Contributor

@mageprince mageprince commented Mar 28, 2019

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 replace deprecated attribute

Additional information

List all affected URLs

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

@devops-devdocs
Copy link
Collaborator

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

@keharper
Copy link
Contributor

Closing because this PR is not correct. See #4064 and #3925

@keharper keharper closed this Mar 28, 2019
@ghost
Copy link

ghost commented Mar 28, 2019

Hi @mageprince, 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.

@mageprince
Copy link
Contributor Author

mageprince commented Mar 29, 2019

@keharper I have replaced a deprecated attribute in develop branch which is created for 2.3.2 and I checked #3925 @mage2pratik has missed adding an attribute so I have created this PR based on his PR. So can you please reopen this and proceed further?

@keharper keharper reopened this Mar 31, 2019
@keharper keharper self-assigned this Mar 31, 2019
@devops-devdocs
Copy link
Collaborator

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

@keharper keharper added xx2.3.1 Magento 2.3.1 changes xx2.3.2 Magento 2.3.2 changes and removed xx2.3.1 Magento 2.3.1 changes labels Mar 31, 2019
@keharper keharper added Technical Updates to the code or processes that alter the technical content of the doc Community Docs impacted by community code contribution labels Apr 23, 2019
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.

Verified this change is valid for 2.3.2.

Listed canonical_url as deprecated and general cleanup.
@keharper keharper merged commit d72419d into magento:develop Apr 23, 2019
@ghost
Copy link

ghost commented Apr 23, 2019

Hi @mageprince, 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

Thanks @mageprince

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Community Docs impacted by community code contribution Partner: Krish TechnoLabs partners-contribution PR created by Magento partner Technical Updates to the code or processes that alter the technical content of the doc xx2.3.2 Magento 2.3.2 changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants