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

Review needed #10

Closed
mattheworiordan opened this issue Mar 25, 2015 · 1 comment
Closed

Review needed #10

mattheworiordan opened this issue Mar 25, 2015 · 1 comment
Labels
content-request A request for new content, as opposed to changing/fixing existing content

Comments

@mattheworiordan
Copy link
Member

@paddybyers and @kouno, please can you review my changes in the docs repo over the last few days, see 8a62a69...source

The biggest chunks of work are:

  • Reorganisation of the client library developer guide sections with lots of new content
  • Support for wider variable interpolation within code examples
  • Update of out of date documentation include Authentication, WebSocket & Comet transports, Encryption, Real-time Protocol, REST API etc. I am focussing on this now because I want to help client lib developers by having docs that are up to date. @paddybyers especially, please can you review my changes as it's quite likely there are still parts that are wrong. I am less worried about missing docs however quite worried if the docs are wrong.
  • I set up the Quick Start Guide as a test to see if I could get the "Try it" live code editor working again properly with our JSBin. This all works now and the workflow is actually quite easy, you can either work in repo itself and the changes to JSBin are pushed up live when running guard, or you can edit in JSbin and then export back to docs by appending .textile to the URL in JSBin.

Please can you review these changes as soon as possible as I will be telling all client library developers that the documentation in those sections is now mostly up to date.

@kouno we are also now ready to get going with integrating our examples into the docs repo.

@mattheworiordan mattheworiordan added the content-request A request for new content, as opposed to changing/fixing existing content label Mar 25, 2015
@mattheworiordan
Copy link
Member Author

N/A

MymmiJ added a commit that referenced this issue Mar 15, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Apr 11, 2022
* CircleCI config for lint and test
DuncanEdwards pushed a commit that referenced this issue Apr 20, 2022
* CircleCI config for lint and test
DuncanEdwards pushed a commit that referenced this issue May 23, 2022
* CircleCI config for lint and test
DuncanEdwards pushed a commit that referenced this issue May 23, 2022
* CircleCI config for lint and test
DuncanEdwards pushed a commit that referenced this issue May 25, 2022
* CircleCI config for lint and test
DuncanEdwards pushed a commit that referenced this issue May 25, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Jun 27, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Sep 23, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Oct 31, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Nov 9, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Nov 15, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Nov 17, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Nov 17, 2022
* CircleCI config for lint and test
MymmiJ added a commit that referenced this issue Dec 5, 2022
* CircleCI config for lint and test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content-request A request for new content, as opposed to changing/fixing existing content
Development

No branches or pull requests

1 participant