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

Knowledge Base #211

Closed
wants to merge 5 commits into from
Closed

Knowledge Base #211

wants to merge 5 commits into from

Conversation

mikeal
Copy link
Contributor

@mikeal mikeal commented Oct 6, 2015

This is a migration of the documentation Nodejitsu built over the last 5 years or so. Before we can merge I'll have to finalize the transfer of copyright from Nodejitsu/GoDaddy to the foundation.

This work would provide a nice starting point for a knowledge base we can continue to build and iterate on as a community.

I also removed author information. IMO having the initial author noted for each article is a barrier to others editing and maintaining those pages.

Things still to do:

  • Improve the template so that it suggests other content.
  • Create a directory page that shows all content.
  • Find a good way to expose and link to this content from the main docs page.

/cc @indexzero @nodejs/documentation

@indexzero
Copy link
Contributor

+1 here on the Copyright transfer. We'll work to get that sorted with legal folks so it can transfer from Nodejitsu to the foundation.

What are thoughts on putting this in a separate repo and including it as a module?

@mikeal
Copy link
Contributor Author

mikeal commented Oct 6, 2015

@indexzero this repo is already setup for auto-deploy and has a localization system attached, and will soon have localizers and doc writers as write access members. It's going to get pretty complicated to do an additional repo with all of that. Not to mention this repo has contributors already making style improvements and whatnot. This is the path of least resistance for sure.

@Qard
Copy link
Member

Qard commented Oct 6, 2015

Great stuff! This'll really help to seed the Docs WG content. I'd be happy to help out with that todo list. 😄

@mikeal
Copy link
Contributor Author

mikeal commented Oct 6, 2015

@Qard go ahead and send pull requests to the knowledge-base branch to help out :)

@indexzero
Copy link
Contributor

@mikeal 10-4. We'll follow-up offline about the next steps w.r.t. copyright transfer. I assume this will be a topic at the November 2nd board meeting?

@fhemberger
Copy link
Contributor

Please also check how this content relates to #217, #218. It's a bit confusing at the moment.

Add auto-generated index to knowledge base
@fhemberger
Copy link
Contributor

@mikeal @indexzero Is there any progress on this? Would love to hand this over to @nodejs/documentation.

@indexzero
Copy link
Contributor

@fhemberger agreed. Nodejitsu would love to hand this over to @nodejs/documentation as well! I believe the next step here was for this to be discussed during the last foundation legal WG call. I do not sit on the legal committee, but I know @mikeal is aware of the minutes. Hopefully he can give status.

Due an unavoidable work conflict at GoDaddy next Monday I will have to miss most of the Node.js Foundation board call.

@mikeal if this will be part of the public or private session I will make myself available at the pleasure of the board while it is discussed.

@mikeal
Copy link
Contributor Author

mikeal commented Oct 28, 2015

hey @indexzero I think it would help if you could add an MIT/BSD compatible license to the original work on https://github.com/nodejitsu/docs/

@indexzero
Copy link
Contributor

@mikeal done. It is now MIT. nodejitsu/docs@a4800be

@mikeal
Copy link
Contributor Author

mikeal commented Oct 28, 2015

werd, ok, we can merge this now. we'll work out the details of the copyright transfer later, now that it's permissively licensed we can put it in the tree.

@indexzero
Copy link
Contributor

Resisting the urge to push the "Merge" button so hard at the moment. All sentimental. Holding off for the final ok from @fhemberger: are we good to go here?

@fhemberger
Copy link
Contributor

@indexzero @mikeal Should we merge it without the Handlebars templates, so @nodejs/documentation can integrate the docs into their content structure first?

I want to avoid adding redirects later.

@indexzero
Copy link
Contributor

@fhemberger Nodejitsu plans to redirect all traffic from http://docs.nodejitsu.com to the final pages on the nodejs.org site so we will have to do the redirect dance a little no matter what.

Obviously once is better than twice, we could move this into a "draft" area if one exists where it would not get rendered or served yet. Given how old these tutorials are that's probably the best way to approach this: stage them somewhere and publish them one at a time as they are reviewed for correctness.

@fhemberger
Copy link
Contributor

Our staging area at the moment is called "push it without linking it on the site" ;)

@indexzero
Copy link
Contributor

@fhemberger great. So this can be safely merged then?

@fhemberger
Copy link
Contributor

@indexzero I had to fix a few things:

  • the missing 'docs-menu' partial in layouts/docs.hbs and layouts/knowledge-base-index.hbs was breaking the the build.
  • I excluded the knowledge base index file from the listing and fixed markdown headlines in two documents
  • I removed "Downloads" (linking to Node.js 0.4.1 ;)) and 'Hosting, Conferences and More!' (we have a event listing already and some links were broken). I hope that's ok for you.

@fhemberger
Copy link
Contributor

Merged via d02b8a2.

@fhemberger fhemberger closed this Oct 29, 2015
@indexzero
Copy link
Contributor

\o/

@indexzero indexzero removed the on hold label Nov 19, 2015
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.

4 participants