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

Epic: collect needed documentation #378

Open
7 of 9 tasks
fyliu opened this issue Sep 7, 2024 · 31 comments · Fixed by #430
Open
7 of 9 tasks

Epic: collect needed documentation #378

fyliu opened this issue Sep 7, 2024 · 31 comments · Fixed by #430
Labels
complexity: missing epic Issue is an epic feature: docs: PD team documentation documentation on PD team processes and architecture, etc. feature: process improvement good first issue Good for newcomers role: technical writing s: PD team stakeholder: People Depot Team size: 0.25pt Can be done in 1.5 hours or less

Comments

@fyliu
Copy link
Member

fyliu commented Sep 7, 2024

Overview

This issue tracks the documentation that needs to be added and the issue/pr where they would've been helpful in. The intention is to make it easy to note any missing documentation without having to create the issue at the same time.

Action Items

Resources/Instructions

How to use

  1. Add a new comment for each documentation item needed using a copy of this template

    ### What's needed
    
    - Description of what the missing documentation is about
    
    ### Draft (if written)
    
    - Link to external docs with instructions (if applicable)
    
    ### Where it's useful
    
    - Link to what prompted the need for this change/addition (if applicable)
    - Situation where this could be used
    
    ### Where to put this
    
    **contributing docs** should go in the `/docs/contributing/` folder or sub-folders:
    - **how to guides** --> `/docs/contributing/howto/`
    - **tutorials** --> `/docs/contributing/tutorial/`
    
    ### Audience
    
    - Who or what role needs to perform this action
    
    ### Tags
    
    - git, django, etc.
    
  2. Add an action item here, linking to the comment below

  3. Create an issue to add the documentation (can be done later by another person)

    • Then, hide the original comment and add the new issue # to the action item
  4. Work on the documentation issue and check off the action item (can be done later by yet another person)

@fyliu fyliu added feature: docs: PD team documentation documentation on PD team processes and architecture, etc. feature: process improvement good first issue Good for newcomers role: technical writing s: PD team stakeholder: People Depot Team size: 0.25pt Can be done in 1.5 hours or less labels Sep 7, 2024
@fyliu fyliu added this to the 5 - Team Workflow milestone Sep 7, 2024
@fyliu fyliu moved this to Ongoing in P: PD: Project Board Sep 7, 2024
@fyliu

This comment was marked as resolved.

@fyliu

This comment was marked as resolved.

@fyliu

This comment was marked as resolved.

@fyliu

This comment was marked as resolved.

@fyliu

This comment was marked as resolved.

@fyliu

This comment was marked as resolved.

@shmonks shmonks added the epic Issue is an epic label Oct 4, 2024
@shmonks shmonks changed the title Collect needed documentation Epic: collect needed documentation Oct 4, 2024
@shmonks

This comment was marked as resolved.

@shmonks shmonks self-assigned this Oct 10, 2024
@shmonks

This comment was marked as outdated.

@shmonks

This comment was marked as resolved.

@shmonks shmonks moved this from Ongoing to 🏗In progress-actively working in P: PD: Project Board Oct 18, 2024
@shmonks

This comment was marked as outdated.

@fyliu

This comment was marked as resolved.

@shmonks

This comment was marked as resolved.

@github-project-automation github-project-automation bot moved this from 🏗In progress-actively working to ✅Done in P: PD: Project Board Nov 8, 2024
@ExperimentsInHonesty

This comment was marked as resolved.

@shmonks shmonks reopened this Nov 8, 2024
@github-project-automation github-project-automation bot moved this from ✅Done to ❓Questions/Review in P: PD: Project Board Nov 8, 2024
@shmonks shmonks moved this from ❓Questions/Review to Ongoing in P: PD: Project Board Nov 8, 2024
@fyliu
Copy link
Member Author

fyliu commented Nov 21, 2024

The contributing docs should go inside the /docs/contributing/ folder or sub-folders

  • howto guides --> /docs/contributing/howto/
  • tutorials --> /docs/contributing/tutorial/

maybe temporary locations

  • style guides --> `/docs/contributing/style/'
    • bash scripting guide (example contents below)
      • set env options for each script (what's most common, what they mean, how to use them)
      • don't leave debug output
    • new documentation page guide

@fyliu
Copy link
Member Author

fyliu commented Nov 21, 2024

What's needed

A guide for creating a new documentation page

I'm not sure if this falls under "style guides" or "howto"

Draft

Tags

  • add tags to each page (example tags, how it works, which ones are strongly recommended like)
Example tags
  1. reader: one of "contributing-docs"/"client-docs"
  2. type: one of "intro"/"howto"/"tutorial"/"style"
  3. topic: up to 5 relevant from "git", "django", etc.
How to add them

create a front matters section at the top of the document like this:

---
tags:
  - contributing-docs
  - howto
  - git
---

File location

  • directory structure

    ├── docs                                                                                                                                                                                                                                       
    │   ├── contributing                                                                                                                                                                                                                           
    │   │   ├── onboarding                                                                                                                                                                                                                
    │   │   ├── tutorials
    │   │   ├── howto
    │   │   ├── style
    │   │   └── tools
    │   ├── intro                
    │   ├── topics
    │   ├── tutorials                
    │   ├── howto                
    │   └── API                                                
    
  • where to put the page

    graph LR
      B -->|contributing| contributing;
      B{contributing/client?} -->|client| client;
    
      subgraph contributing [contributing docs]
      direction LR
      C -->|onboarding| F[/docs/contributing/onboarding/];
      C -->|tutorial| D[/docs/contributing/tutorials/];
      C -->|how to| E[/docs/contributing/howtos/];
      C -->|style guide| G[/docs/contributing/style/];
      C{content type} -->|tools| A[/docs/contributing/tools/];
      end
    
      subgraph client [client docs]
      direction LR
      H -->|introductory| I[/docs/intro/];
      H -->|high level description| M[/docs/topics/];
      H -->|tutorial| J[/docs/tutorials/];
      H -->|howto| K[/docs/howto/];
      H{content type} -->|API| L[/docs/api/];
      end
    
    
    Loading

Voice and tone

  • voice style (suggestion until we have a technical writer to figure this out)
    • howto guide: conversational tone - "you do this", "you'll see this"
    • tutorial: "we" tone. Think of it as a longer journey - "we need to set this up", "now we're finally done"
    • TBD

TBD

Where it's useful

Someone adding a new documentation page can reference this to make sure they're making use of all the available functionalities of the docs site and the documentation is like the rest of the documentation.

There's a comment asking where new docs pages should go. That's also included in this guide.

Audience

contributors, technical writers

Tags

howto, style guide, documentation

@shmonks
Copy link
Member

shmonks commented Nov 21, 2024

@fyliu Do you think the guide for creating a new documentation page be best in Wiki or Mkdocs?

@fyliu
Copy link
Member Author

fyliu commented Nov 21, 2024

@shmonks I was thinkin Mkdocs. Maybe it's helpful to keep it in the wiki until it's more stabilized?

Oh I see what you mean. This might be for non-developers as well. But this guide is for adding documentation to the Mkdocs site. So I'm thinking the writer will have Mkdocs running locally on their computer. But I guess the writer can also modify the text in GitHub's website and create the PR, which would automatically fix any syntax problems in the markdown, and then another writer can review and approve.

I think most of the howto guides actually needs someone with a developer background to go through them to make sure they produce the expected outcome.

What should be the process of working on an approving this type of work? Developer create the first draft and writer review and produce a final one?

@shmonks shmonks removed their assignment Nov 22, 2024
@shmonks
Copy link
Member

shmonks commented Nov 22, 2024

Yes, I imagine we'd want a review - via PR probably best, then we could assign it to whoever. It sounds like Mkdocs would be the best place to have this new guide, with a link to it from the wiki.

@fyliu
Copy link
Member Author

fyliu commented Nov 22, 2024

Once this is created, then update the following template
https://github.com/hackforla/peopledepot/blob/main/.github/ISSUE_TEMPLATE/update-table.md

What's needed

  • How to generate a database table definition

Draft

Note: These steps work only for this project's database running locally in Docker container.

Shorter way

  1. Get table definition

    ./scripts/db.sh -c "\d <table_name>"
    

    Example table name: core_user

  • (Optional) Get all table names

    ./scripts/db.sh -c "\dt"
    

Long way

  1. Enter the database shell in the docker container

    ./scripts/db.sh
    
  2. Get list of tables

    \dt
    
  3. Get table definition

    \d <table_name>
    

Where the documentation needs to go

  • /docs/contributing/howto/generate-db-definition.md

Where it's useful

  • Update Table: User #429 needed a way to generate a schema table in text format.
  • Other similar issues can also use this guide.
  • Link to what prompted the need for this change/addition
  • Situation where this could be used

Audience

  • contributor, developer

Tags

  • postgres, schema, database

@fyliu fyliu mentioned this issue Nov 22, 2024
8 tasks
@fyliu
Copy link
Member Author

fyliu commented Nov 22, 2024

What's needed

  • MD file syntax guide

Draft (very incomplete at the moment)

code block indentation

  • if under a header, no indent
  • if under a bullet line, indent entire block 3 or 4 spaces in GitHub, or 4 spaces in MkDocs markdown

lists

  • bullet lists start with -. Standardizing so we don't have a mix of - and * bullets.
  • numbered lists start with 1.. The numbers will be autogenerated by the markdown renderer.

Where it's useful

Where to put this

Most of the contents are for the MkDocs site, so /docs/contributing/howto/format-markdown.md
But some contents are useful for GitHub as well, so also should add a link from the Wiki.

Audience

  • contributors

Tags

  • markdown

@shmonks
Copy link
Member

shmonks commented Jan 3, 2025

What's needed

  • Add troubleshooting section for GitHub SSH keys (for first time users or new machine).

Draft (if written)

Where it's useful

  • Link to what prompted the need for this change/addition (if applicable)
  • Situation where this could be used

Where to put this

contributing docs should go in the /docs/contributing/ folder or sub-folders:

  • how to guides --> /docs/contributing/howto/
  • tutorials --> /docs/contributing/tutorial/

Audience

  • Who or what role needs to perform this action

Tags

  • git, django, etc.

@fyliu
Copy link
Member Author

fyliu commented Jan 24, 2025

@shmonks
commenting on the SSH comment #378 (comment) above:
The website guide looks useful and has instructions for mac and windows.

There's benefits both for putting it in our own mkdocs vs. Engineering Cop. With the tabs widget in mkdocs, it would make the instructions look shorter (hiding the instructions for the other OS). But putting it in Engineering CoP wiki would centralize it for other teams to reference, which is also a plus.

I think the best scenario would be to put it in Engineering CoP in mkdocs. We'd need to set it up for them (which is quick, and a one time thing) and consider what, if any, maintenance they'll have to do to mkdocs other than just writing the documentation in markdown.

@del9ra
Copy link
Member

del9ra commented Jan 31, 2025

What's needed

  • After the recent Docker updates, we need to replace the hyphen with a space in commands starting with docker-compose in script files like run.sh and migrate.sh. The hyphen is no longer needed. I encountered an issue where the docker-compose command didn’t work when I ran ./scripts/buildrun.sh

Draft (if written)

https://docs.docker.com/compose/releases/migrate/

Where it's useful

Where to put this

https://hackforla.github.io/peopledepot/contributing/dev_environment/#install-docker
https://github.com/hackforla/website/blob/gh-pages/CONTRIBUTING.md#16-dev-setup-6-build-and-serve-the-website-locally
https://github.com/hackforla/website/blob/gh-pages/CONTRIBUTING.md#i-a-few-notes-regarding-docker

Audience

Backend developer

Tags

  • git, django, etc.

@fyliu
Copy link
Member Author

fyliu commented Feb 1, 2025

  • After the recent Docker updates, we need to replace the hyphen with a space in commands starting with docker-compose in script files like run.sh and migrate.sh. The hyphen is no longer needed.

@del9ra What do you think about updating all the calls to use the new syntax in the scripts?

They made the change maybe 2+ years ago. But some recent updates starting maybe a year ago started breaking the docker-compose command.

@del9ra
Copy link
Member

del9ra commented Feb 3, 2025

  • After the recent Docker updates, we need to replace the hyphen with a space in commands starting with docker-compose in script files like run.sh and migrate.sh. The hyphen is no longer needed.

@del9ra What do you think about updating all the calls to use the new syntax in the scripts?

They made the change maybe 2+ years ago. But some recent updates starting maybe a year ago started breaking the docker-compose command.

@fyliu Up until now, when running ./scripts/buildrun.sh, I only had issues with run.sh and migrate.sh. But if you've run into the same problem with other scripts, then yes, it makes sense to update all the calls. By the way, I didn’t have any issues with docker-compose command last year. The problems only started this January.

@fyliu
Copy link
Member Author

fyliu commented Feb 3, 2025

@fyliu Up until now, when running ./scripts/buildrun.sh, I only had issues with run.sh and migrate.sh. But if you've run into the same problem with other scripts, then yes, it makes sense to update all the calls. By the way, I didn’t have any issues with docker-compose command last year. The problems only started this January.

@del9ra not all newer versions will break it. Only some updated do. I've experience it twice before, starting maybe a year ago, but not currently.

@fyliu
Copy link
Member Author

fyliu commented Feb 7, 2025

In response to my comment above about moving common documentation to Engineering CoP, since they're not working on adding engineering documentation to even their wiki, I guess we'll just have to add what our team needs in our own docs for now. We can reference the website wiki where it's helpful. Maybe we should also make a local copy of the original text just in case something happens to the source page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: missing epic Issue is an epic feature: docs: PD team documentation documentation on PD team processes and architecture, etc. feature: process improvement good first issue Good for newcomers role: technical writing s: PD team stakeholder: People Depot Team size: 0.25pt Can be done in 1.5 hours or less
Projects
Status: Ongoing
Development

Successfully merging a pull request may close this issue.

4 participants