Opened 15 months ago
Last modified 15 months ago
#34954 assigned Cleanup/optimization
Improve `output_field` resolution of `Concat`
Description ¶
Simon proposed in this comment to release blocker issue #34944 , to open a new improvement issue for a version of Django after 5.0
Replying to Simon Charette
FWIW I create a draft MR to demonstrate how
Concat
resolving could be enhanced but I still think that we should not included it in 5.0. Might be worth a follow up cleanup/optimization ticket though.
According to the ticket's flags, the next step(s) to move this issue forward are:
- To provide a patch by sending a pull request. Claim the ticket when you start working so that someone else doesn't duplicate effort. Before sending a pull request, review your work against the patch review checklist. Check the "Has patch" flag on the ticket after sending a pull request and include a link to the pull request in the ticket comment when making that update. The usual format is:
[https://github.com/django/django/pull/#### PR]
.
Change History (2)
comment:1 by , 15 months ago
Cc: | added |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 15 months ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Summary: | Improve `output_field` resolution in `GenerateField` → Improve `output_field` resolution of `Concat` |
Note:
See TracTickets
for help on using tickets.
Renaming the ticket as it cannot be done generically for
GeneratedField
. Each expression must be adapted to do a better job at dealing with mixed input and my comment was solely for concat which is straight forward to resolve.