Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Documentation: Consolidate Technical Documentation #360

Closed
8 tasks done
allella opened this issue Mar 6, 2020 · 14 comments
Closed
8 tasks done

Documentation: Consolidate Technical Documentation #360

allella opened this issue Mar 6, 2020 · 14 comments
Assignees
Labels
Documentation Improvements or additions to documentation Good First Issue This issue is beginner friendly! It shouldn't take much experience to get up and running.

Comments

@allella
Copy link
Contributor

allella commented Mar 6, 2020

Quincy suggested all technical docs go into CONTRIBUTING.md on the Jan 25 2020 meeting.

@allella allella added the Discussion Ideas, feature requests, views on features. Anything which is a discussion. label Mar 6, 2020
@allella
Copy link
Contributor Author

allella commented Mar 6, 2020

The /docs/images and /docs/how-to-open-a-pull-request.md are used or linked to from within the CONTRIBUTING.md, so we don't want to delete them without consolidating.

@allella
Copy link
Contributor Author

allella commented Mar 6, 2020

  1. This also begs the question of if some of the README.md technical documentation is better in CONTRIBUTING.md.

These sections of the README may be better in CONTRIBUTING.md

@Zeko369 @timmyichen would you agree on moving these sections and/or any others?

@allella allella added Documentation Improvements or additions to documentation Beginner Friendly and removed Discussion Ideas, feature requests, views on features. Anything which is a discussion. labels Mar 6, 2020
@Zeko369
Copy link
Member

Zeko369 commented Mar 7, 2020

@allella Thank for openinig this PR and bringing this up. And sorry for the late replay I had a lot of stuff to do yesturday.

  1. I was working on something but don't have a PR open, I'll do it with @Narutuffy.
  2. Agreed, but we should also add how to pull other peoples branches, and maybe how to push to them (this can only be done by people with maintainter or higher level access to this repo, but seems important non the less)
  3. Agreed
  4. I'd keep it in docs/images or maybe rename it to docs/assets
  5. Yes, that would be quite neat
  6. I think that @Narutuffy should rewrite parts of the docs, or at least just do this migration
  7. I think that @Narutuffy should rewrite parts of the docs, or at least just do this migration
  8. We should move that to the CONTRIBUTING.md, but there are some things about them that need to be changed, for example the Schema image generation Schema image / ER Diagram is out of date #54, API (swagger.json) docs need to be heavily refactored.

@allella
Copy link
Contributor Author

allella commented Mar 8, 2020

Regarding 4, how about we create an
/assets
directory and move everything out of the /docs directory?

@Zeko369 I assume there will be other assets for the front end. Is there a pattern from another project we'd like to use to store web app assets, including the limited number of docs images?

@Zeko369
Copy link
Member

Zeko369 commented Mar 11, 2020

@allella For the next.js all the assets are in the /public folder, and I don't really think we should put docs images there, I'd keep it in docs/assets so we can differentiate them from the app assets.

@allella
Copy link
Contributor Author

allella commented Mar 11, 2020

Alright, @Narutuffy, are you still available to help with this issue? Many of the steps are mentioned above, but I can assist and confirm.

@Narutuffy
Copy link
Contributor

Narutuffy commented Mar 12, 2020

Yeah definitely, I will ping you and confirm for overall clarity @allella I'll start working on a branch.

@allella
Copy link
Contributor Author

allella commented Mar 13, 2020

@Narutuffy I've updated this issue with a checklist and you knocked out 2-6. If you can help with 1,7,8, then I suspect these tasks and conversations should lead to other ways you can contribute on the more technical aspects.

  1. is related to Schema image / ER Diagram is out of date #54 because the issue was re-opened to talk about automatically generating a schema E-R Diagram image so the documentation is always updated when the schema changes. @Zeko369 worked with I just commented that it looks like Github Pages would allow us to host the /docs directory, which could potentially allow us to host the interactive SchemaSpy files at a URL like https://freecodecamp.github.io/chapter/schema

@allella
Copy link
Contributor Author

allella commented Mar 13, 2020

@Narutuffy this is simple, but the following images can be deleted. They were part of an issue and I moved the screenshots into that issue.

  • docs/assets/GoogleGeneratedSnippet.png
  • docs/assets/MeetupGroupEventSERBreadcrumbs.png
  • docs/assets/RegularSER.png

@allella
Copy link
Contributor Author

allella commented Mar 13, 2020

@Zeko369 @nik-john I seem to recall the Swagger documentation was hosted on a custom HTTP port, like http://localhost:8000/api/v1/docs

I don't know if this is a good idea, or even possible, but could we export Swagger HTML files to /docs/api automatically so they can be viewed through the GitHub pages? Or, is it best to leave it as-is and expect developers to visit the custom HTTP port to see those docs?

@allella
Copy link
Contributor Author

allella commented Mar 13, 2020

@Narutuffy @Zeko369 for task 8) on this issue, I think as a first step we should consolidate all the remaining sections we want into CONTRIBUTING.md and then talk about refactoring them.

I think if we wait until everything has been refactored, then we'll delay and complicate an otherwise easy copy / paste and reformat.

Does everyone agree to move the following sections into CONTRIBUTING.md in their current state? Or, would we like to keep any of the README.md sections below in the README?

@allella
Copy link
Contributor Author

allella commented Apr 1, 2020

@Narutuffy I hope things are safe and stable in your life and community.

I'm going to have time to contribute more on the documentation. I can continue with the remaining steps, but if you have time to continue, then please let me know to avoid duplicating efforts.

Thanks

@allella allella self-assigned this Jul 21, 2020
@allella
Copy link
Contributor Author

allella commented Jul 21, 2020

I'm going to continue this issue and consolidate the existing technical documents (1 and 8 above) into CONTRIBUTING.md and we can continue updating the Docker section and others once it's all in one place.

Quincy shared the freeCodeCamp contributing domain / page as an example of having things in one place. This is a first step in that direction.

@allella
Copy link
Contributor Author

allella commented Aug 15, 2020

#1 from the list above is targeted by #397

@allella allella closed this as completed Mar 8, 2021
@allella allella added Good First Issue This issue is beginner friendly! It shouldn't take much experience to get up and running. and removed Beginner Friendly labels Aug 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Documentation Improvements or additions to documentation Good First Issue This issue is beginner friendly! It shouldn't take much experience to get up and running.
Projects
None yet
Development

No branches or pull requests

3 participants