Changes between Initial Version and Version 1 of Ticket #36702, comment 3


Ignore:
Timestamp:
Nov 2, 2025, 7:03:11 PM (3 weeks ago)
Author:
Simon Charette

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #36702, comment 3

    initial v1  
    11Thanks for the test!
    22
    3 I think we should merge your proposed `if field != opts.pk:` removal with your test first though as my branch is more invasive and this ticket is already focused on it.
     3I think we should merge your proposed `if field != opts.pk:` removal with your test first though as my branch is more invasive and this ticket is already focused on a particular accepted bug while the other is more of an optimization.
    44
    5 Would you mind pushing the removal to your branch and open a PR with it? Happy to review it one you do and we can trade reviews when I open the follow up ticket and PR to optimize the insertion.
     5Would you mind pushing the removal to your branch and open a PR with it? Happy to review it one you do and we can trade reviews when I open the follow up ticket and PR to optimize the insertion if you're interested.
Back to Top