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

Feedback and Additions on Azure Functions walkthrough #1

Merged
merged 1 commit into from
Feb 11, 2019
Merged

Feedback and Additions on Azure Functions walkthrough #1

merged 1 commit into from
Feb 11, 2019

Conversation

michael-watson
Copy link

Hi Erick, thanks for your hard work on this. Over the weekend I walked through everything and added debugging/deployment for VS Code. I made some small tweaks to standardize things, but most of the changes were minor.

  • Add missing Functions CLI as pre-requisite
  • Remove numbered versioning of CLI - although this is helpful, it is difficult to maintain the doc
  • Re-structure deploment setup of docs - This is to add in VS Code stuff
  • Added documentation on how to have url be http://localhost:7071/graphql
  • Changed Apollo Server sample code to be from our getting started

Let me know if you have any questions, happy to talk about any changes you think should be made! I should be able to get some members from the team to review once you have accepted these changes.

Add missing Functions CLI as pre-requisite
Remove numbered versioning of CLI - although this is helpful, it is difficult to maintain the doc
Re-structure deploment setup of docs - This is to add in VS Code stuff
Added documentation on how to have url be http://localhost:7071/graphql
Changed Apollo Server sample code to be from our getting started
@ErickWendel
Copy link
Owner

Hi @michael-watson thanks for the feedback 😄

At my examples, I tried to maintain the same example that I saw before on the Apollo Server website. At this PR you change from our object list to a string hello world's example. I think that is good to the reader just focus on the Apollo features, so I agree with you

@ErickWendel ErickWendel merged commit 1e74d6e into ErickWendel:add/azure-function-examples Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants