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

Clarify PHP version support and fix format errors #7042

Merged
merged 4 commits into from
Apr 16, 2020
Merged

Conversation

meker12
Copy link
Contributor

@meker12 meker12 commented Apr 15, 2020

Purpose of this pull request

Fix a formatting issue in PHP version constraint information for URL rewrites module.

  • The | character in the version constraints caused issues rendering the callout.
  • Changed PHP version support from annotation to English

Affected DevDocs pages

- The | character in the version constraints caused issues rendering the callout.
-  Changed PHP version support from annotation to English
@meker12 meker12 added 2.x Internal Dev Differentiates work between community and Magento staff Small Changes labels Apr 15, 2020
@meker12 meker12 self-assigned this Apr 15, 2020
@meker12 meker12 requested a review from hguthrie April 15, 2020 16:39
@@ -10,7 +10,7 @@ functional_areas:
You can easily migrate to the {{site.data.var.ece}} platform without losing SEO rankings and traffic. Use the `magento/url-rewrite-import-export` module to redirect traffic from your old, indexed URLs to new URLs.

{:.bs-callout-info}
This module supports ~7.0.13|~7.1.0|~7.2.0|~7.3.0 PHP versions and is available for Magento version 2.2.x and 2.3.x only.
This module supports the following PHP versions 7.0.13 and later patch releases and all 7.1.0 and 7.2.0 patch releases. The module is available for Magento version 2.2.x and 2.3.x only.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this be better by removing "the following"?

@meker12
Copy link
Contributor Author

meker12 commented Apr 16, 2020

running tests

@meker12 meker12 merged commit 9a01d09 into master Apr 16, 2020
@ghost
Copy link

ghost commented Apr 16, 2020

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

@meker12 meker12 deleted the mae-fix-format-issue branch April 16, 2020 21:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Internal Dev Differentiates work between community and Magento staff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants