Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed Doc Block for the dispatch method of the Rest Controller #8677

Merged
merged 1 commit into from
Feb 25, 2017
Merged

Fixed Doc Block for the dispatch method of the Rest Controller #8677

merged 1 commit into from
Feb 25, 2017

Conversation

vrann
Copy link
Contributor

@vrann vrann commented Feb 25, 2017

During MageTitans Hackathon in Mexico it was noticed that the dispatch method of the Rest Controller does not provide enough information in the description on how the method is supposed to work. The description was updated to highlight the key points of method execution. This information will help developers to understand the method better. It will be used by automatic documentation generation tools.

The content of the Pull Request:

  • updated doc block for the method dispatch in the Rest Controller.

- provided more information on how dispatch method handles the Request
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Feb 25, 2017

CLA assistant check
All committers have signed the CLA.

@okorshenko okorshenko self-assigned this Feb 25, 2017
@okorshenko okorshenko added this to the February 2017 milestone Feb 25, 2017
@okorshenko
Copy link
Contributor

@vrann thank you for your contribution. Could you please sign CLA?

@vrann vrann self-assigned this Feb 25, 2017
@okorshenko
Copy link
Contributor

@vrann thank you for your contribution. Your pull request will be merged in a few minutes

@mmansoor-magento mmansoor-magento merged commit f0d99be into magento:develop Feb 25, 2017
magento-devops-reposync-svc pushed a commit that referenced this pull request Dec 15, 2023
…elivery

[LYNX] Storefront extra graphql delivery
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants