Need (editable) display name for Target

[phraenquex]

Display name (in landing page) needs to be changeable, after the first upload.

So needs "display_name" alongside "name". Small change to model, says Tim.

Needs a Modal for editing the name. Any authorised user.

(Might duplicate another ticket... @phraenquex to find.)

[kaliif]

Patch method added to api/targets, Allows editing target object's display_name attribute (only that, nothing more). In staging as of 2.0.36

[mwinokan]

@boriskovar-m2ms, @matej-vavrek: have you tried the new endpoint?

[matej-vavrek]

I have tried it. It seems working as api/targets/{targetId}/

[phraenquex]

Terrific - now we just need a frontend mechanism to edit it. I suggest an item on the hamburger menu, call it like "Edit target info".

Pop up a modal with "Target display name".

(In future, we'll use the modal for additional metadata.)

@mwinokan please critique/confirm.

[boriskovar-m2ms]

Any update on this? May I implement it the way @phraenquex described in previous comment?

[phraenquex]

@boriskovar-m2ms go ahead according to my spec

[boriskovar-m2ms]

In scope are also changes that whenever the name of the target is displayed to the user it should consistently display field display_name and not title. Currently we are only using title field in the code.

[Waztom]

@boriskovar-m2ms talked me through the changes - state does not reload on Target name change and the state is repainted in a way that leads to irregular behaviour. Estimate > 1 week of work to fix. Alternative, is to edit the target name via the Target Name list. This will solve state challenge and be a few hours of work.

[boriskovar-m2ms]

Main culprit for this erratic behavior is how are we handling NGL view. The problem is that NGL view is not reacting to our state but we are imperatively controlling the NGL view so when our state change requires remounting of NGL view component it will be not redrawn correctly. This is something that #1483 also addressing.