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

Packages.md #5908

Merged
merged 3 commits into from
Nov 1, 2019
Merged

Packages.md #5908

merged 3 commits into from
Nov 1, 2019

Conversation

dewimagento
Copy link
Contributor

Purpose of this pull request

This PR updates the packages.md file:

  • Fix "smart" unicode quotes throughout to correctly be the " and ' characters.
  • Wrap lines to ~120 chars throughout.
  • Remove all references to the DELETE package endpoint, no longer available to users
  • Revamped the parameter reference table; added missing params, renamed obsolete ones, added a column for filterability, and one for the valid values. Worried this might make it look very wide or word-wrapped, my previewer isn't great :( But that'll be cosmetic, can fix later. The data is there, now.
  • Added missing object definitions (browsers, technical options)
  • Minor clarifications and corrections on things like the behavior of validation.

Affected DevDocs pages

https://devdocs.magento.com/guides/v2.3/marketplace/eqp/packages.html

@devops-devdocs
Copy link
Collaborator

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

@dobooth dobooth added 2.3.x Magento 2.3 related changes Internal Dev Differentiates work between community and Magento staff Technical Updates to the code or processes that alter the technical content of the doc and removed Progress: approved labels Nov 1, 2019
@dobooth dobooth merged commit fb1960e into magento:eqp-integration Nov 1, 2019
@ghost
Copy link

ghost commented Nov 1, 2019

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

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 Internal Dev Differentiates work between community and Magento staff 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.

3 participants