Opened 9 months ago

Closed 9 months ago

Last modified 9 months ago

#22906 closed New feature (fixed)

Give ModelState a __repr__()

Reported by: CollinAnderson Owned by: nobody
Component: Migrations Version: 1.7-beta-2
Severity: Normal Keywords:
Cc: cmawebsite@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I've gotten a number of InvalidBasesError("Cannot resolve bases for %r" % new_unrendered_models) errors. Could we make a repr for ModelState objects?

Change History (5)

comment:1 Changed 9 months ago by CollinAnderson

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Summary changed from ModelState.__repr__ to Give ModelState a __repr__()
  • Type changed from Uncategorized to New feature

comment:2 Changed 9 months ago by claudep

  • Needs tests set
  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 9 months ago by Loic Bistuer <loic.bistuer@…>

  • Resolution set to fixed
  • Status changed from new to closed

In 2572c07cc64177e985b453d0f0b819e85fe080cc:

Fixed #22906 -- Added a more helpful repr to migrations' ModelState.

Thanks Collin Anderson for the report and original patch.

comment:4 Changed 9 months ago by Loic Bistuer <loic.bistuer@…>

In a1ddfe440151c81898b94df4d5ea99696398c3f7:

[1.7.x] Fixed #22906 -- Added a more helpful repr to migrations' ModelState.

Thanks Collin Anderson for the report and original patch.

Backport of 2572c07cc6 from master

comment:5 Changed 9 months ago by CollinAnderson

This is perfect. Thanks so much all.

Note: See TracTickets for help on using tickets.
Back to Top