Opened 3 years ago

Closed 3 years ago

#33966 closed New feature (fixed)

Add support for using KeyTextTransform from lookup

Reported by: AllenJonathan Owned by: AllenJonathan
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords:
Cc: Simon Charette 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 (last modified by AllenJonathan)

Currently, we use nested objects to transform through a JSON field. For example, for
json = {'a': {'b': 'boat'}}

Here, to get 'boat` in text form we use:-
KeyTextTransform(KeyTextTransform('a', 'json_field'), 'b')
This would return 'boat' in text form.

A better UX would be:
KeyTextTransform('json_field__a__b')

Change History (6)

comment:1 by Mariusz Felisiak, 3 years ago

Cc: Simon Charette added
Description: modified (diff)
Triage Stage: Unreviewed → Accepted

comment:2 by AllenJonathan, 3 years ago

Description: modified (diff)

comment:3 by Mariusz Felisiak, 3 years ago

Has patch: set
Needs tests: set
Owner: changed from nobody to AllenJonathan
Patch needs improvement: set
Status: new → assigned

​PR

comment:4 by Mariusz Felisiak, 3 years ago

Needs documentation: set

comment:5 by Mariusz Felisiak, 3 years ago

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: Accepted → Ready for checkin

comment:6 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

Resolution: → fixed
Status: assigned → closed

In 10178197:

Fixed #33966 -- Added support for using KeyTextTransform from lookup.

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