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

Add missing documentation on how to combine migrations in Django #421

Open
3 of 14 tasks
shmonks opened this issue Oct 24, 2024 · 2 comments
Open
3 of 14 tasks

Add missing documentation on how to combine migrations in Django #421

shmonks opened this issue Oct 24, 2024 · 2 comments
Labels
complexity: small All steps are laid out in detail so that someone new to the project can work on it dependency Issue has dependencies feature: docs: PD team documentation documentation on PD team processes and architecture, etc. ready for dev lead ready for product role: back end role: dev role: product s: PD team stakeholder: People Depot Team size: 0.25pt Can be done in 0.5-1.5 hours
Milestone

Comments

@shmonks
Copy link
Member

shmonks commented Oct 24, 2024

Dependency

  • Missing information about how to combine migrations in Django

Overview

We need to make it easy to:

  • Record any missing documentation about combining migrations in Django
  • Keep those records in one place
  • Create new issues based on those records

This issue is the place to record missing documentation and start creating related new issues.

Details

  • We are striving to write documentation that helps developers use Django. See Resource 1.01.
  • This issue tracks the documentation that needs to be added for combining migrations in Django. If you've noticed missing documentation about other Git and/or Django processes,
see the following issues instead

Action Items

  • For each piece of missing information, copy and paste the following template into a new comment below.
### Overview
We need to add [REPLACE WITH WHAT NEEDS ADDING], so that developers can [REPLACE WITH BENEFIT]

### Action Items
- [ ] [REPLACE WITH ACTION ITEMS]

### Resources/Instructions/Tags 
- 1.01 #282 
- 1.0x [REPLACE WITH LINKS TO ANY EXTERNAL DOCS THAT MIGHT BE USEFUL (WITH INSTRUCTIONS, IF NEEDED)]
- 1.0x [REPLACE WITH ANY RELEVANT TAGS e.g., git, django, etc.]

### Documentation Addition Notes
#### What's missing?
[REPLACE WITH DESCRIPTION OF WHAT'S MISSING FROM THE DOCUMENTATION]
   
#### Where should it go?
[REPLACE WITH URL OR INDICATE NEW PAGE AND PARENT PAGE]

####  When would this information have been useful?
[REPLACE WITH LINK TO THE PR/ISSUE/SITUATION THAT PROMPTED THE NEED FOR THIS CHANGE/ADDITION]
   
#### Which roles will benefit most from this information?
[REPLACE WITH ROLES]

  • Fill out as many of the template's fields as possible.
  • Copy and paste a link to your comment in Section 2: List of comments/issues about missing documentation for combining Django migrations below, using the next available number
  • Copy the content of your comment
  • Create a new issue, pasting the content of your comment into the new issue
  • Add the following labels to the new issue
    • feature: docs: PD team documentation
    • size: 0.25pt
    • s: PD team
    • ready for dev lead
    • choose a complexity label that fits
    • choose the role label that fits
  • When this new issue has been created, add a link to it in Section 2: List of comments/issues about missing documentation for combining Django migrations below. Make sure the link to your new issue appears beneath the link to your comment, so we can ensure that all comments get made into issues
  • Hide the original comment (you can mark it 'Resolved')

Resources/Instructions

Section 2: List of comments/issues about missing documentation for combining Django migrations

@shmonks shmonks removed their assignment Oct 24, 2024
@shmonks

This comment was marked as outdated.

@shmonks shmonks added dependency Issue has dependencies and removed draft This issue is not fully-written good first issue Good for newcomers labels Oct 24, 2024
@shmonks shmonks moved this from 🆕New Issue Review to 🧊Ice Box in P: PD: Project Board Oct 25, 2024
@vanessaavviles vanessaavviles self-assigned this Oct 28, 2024
@vanessaavviles vanessaavviles moved this from 🧊Ice Box to 🏗In progress-actively working in P: PD: Project Board Oct 28, 2024
@vanessaavviles vanessaavviles removed their assignment Oct 28, 2024
@vanessaavviles vanessaavviles moved this from 🏗In progress-actively working to 🧊Ice Box in P: PD: Project Board Oct 28, 2024
@shmonks shmonks self-assigned this Oct 28, 2024
@shmonks shmonks added the draft This issue is not fully-written label Oct 28, 2024
@shmonks shmonks moved this from 🧊Ice Box to 🏗In progress-actively working in P: PD: Project Board Oct 28, 2024
@shmonks shmonks removed their assignment Oct 31, 2024
@shmonks shmonks moved this from 🏗In progress-actively working to 🧊Ice Box in P: PD: Project Board Oct 31, 2024
@shmonks shmonks moved this from 🧊Ice Box to 📋Prioritized Backlog in P: PD: Project Board Oct 31, 2024
@shmonks shmonks added ready for product role: product and removed draft This issue is not fully-written labels Oct 31, 2024
@shmonks
Copy link
Member Author

shmonks commented Oct 31, 2024

Overview

We need to add a description of how to combine migrations in Django so that developers can integrate changes effectively. For example, if max_migration.txt is at 0027_... and you want to combine 0025_, 0026_, 0027_ into one.

Action Items

Resources/Instructions/Tags

Documentation Addition Notes

What's missing?

  • The process for combining migrations in Django is:
  1. Roll back core app migrations to 0024

    ./scripts/migrate.sh core 0024
  2. Delete unwanted migration files

    rm app/core/migrations/00{25,26,27}_*
  3. Recreate max_migration.txt for core app

    docker-compose exec web python manage.py create_max_migration_files --recreate core
    # the file core/max_migration.txt should contain "0024_..."
  4. Create combined migration file for core app

    ./script/migrate.sh core

Where should it go?

[REPLACE WITH URL OR INDICATE NEW PAGE AND PARENT PAGE]

Where would this information have been useful?

  • Many initial passes for PRs have multiple migration files that should really be a single Create Table migration

Which roles will benefit most from this information?

  • Back end/Dev

@vanessaavviles vanessaavviles self-assigned this Oct 31, 2024
@vanessaavviles vanessaavviles moved this from 📋Prioritized Backlog to 🏗In progress-actively working in P: PD: Project Board Oct 31, 2024
@shmonks shmonks moved this from 🏗In progress-actively working to 🧊Ice Box in P: PD: Project Board Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: small All steps are laid out in detail so that someone new to the project can work on it dependency Issue has dependencies feature: docs: PD team documentation documentation on PD team processes and architecture, etc. ready for dev lead ready for product role: back end role: dev role: product s: PD team stakeholder: People Depot Team size: 0.25pt Can be done in 0.5-1.5 hours
Projects
Status: 🧊Ice Box
Development

No branches or pull requests

2 participants