Opened 4 years ago
Closed 4 years ago
#31645 closed Cleanup/optimization (fixed)
Models have changes red alert message should say which apps/models
| Reported by: | John Vandenberg | Owned by: | Chinmoy |
|---|---|---|---|
| Component: | Migrations | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Keryn Knight | Triage Stage: | Ready for checkin |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
When running 'migrate', the message appearing when there are missing migrations should say which app/model it detected has missing migrations.
The message is currently:
Your models have changes that are not yet reflected in a migration, and so won't be applied.
Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.
Attachments (1)
Change History (15)
comment:1 by , 4 years ago
| Cc: | added |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:2 by , 4 years ago
| Component: | Uncategorized → Migrations |
|---|---|
| Type: | Uncategorized → Cleanup/optimization |
| Version: | 3.0 → master |
comment:3 by , 4 years ago
Hey I'm a beginner in opensource and would like to work on this issue. May I claim the ticket ?
comment:5 by , 4 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:6 by , 4 years ago
What would be an appropriate message here? Will simply printing out the changes dict work?
comment:7 by , 4 years ago
I think a message similar to that given by makemigrations --dry-run would be good enough?
comment:10 by , 4 years ago
| Has patch: | set |
|---|
comment:12 by , 4 years ago
| Needs tests: | set |
|---|---|
| Patch needs improvement: | set |
comment:13 by , 4 years ago
| Needs tests: | unset |
|---|---|
| Patch needs improvement: | unset |
| Triage Stage: | Accepted → Ready for checkin |
Note:
See TracTickets
for help on using tickets.
Tentatively marking this as accepted (if it's not too invasive to surface) because there's prior acceptance for similar in #25855.