•  
     
    story #24969 [modal] add and remove links (replace current artifact link)
Summary
Empty
[modal] add and remove links (replace current artifact link)
Empty

The field can now be used to add & remove links:

  • Add "Unlink" in the table for existing links
  • Allow to link:
    • without type ("is linked to")
    • or "is child of" (to allow parent selection)
      • Warning: there can be only one selected parent so the "is child of" option can't be selected twice
  • "Add link" adds links one by one. Added links:
    • are appended at the bottom of the list with a green background
    • can be removed directly using the "Remove" button
  • Selector allow to pick-up
    • Link to an id : the picker returns as a result the artifact corresponding to the id (GET request triggered after delay). If no artifact is returned, the picker shows a "No result found message"
    • Possible parents, either by filtering on their name or their id. The possible parents list is loaded when "is child of" is selected and the picker is open

Given there was already an existing link (not marked for removal) to artifact A or a newly added link to artifact A, when artifact A is listed in the artifact picker, it will be disabled and cannot be added again. This prevents situations where user links twice artifact A : the last link "wins" and erases the type of the previous link, which is probably unexpected by the user. This also prevents link "cycles" where artifact A ⇄ artifact B, for example artifact A is child of artifact B who is child of artifact A. If such situations are desirable, we need to gather use-cases before allowing them. See mockups below

At this stage, the field replace the existing "artifact link field" in the modal by default for everyone

Mockups

Artifact picker mockup for this story is available in this figma: https://www.figma.com/file/f8zfcbUP4bpqqs7MB2IsqQ/New-Artifact-Links?node-id=0%3A1

Links list mockup for this story is available in this codepen : https://cdpn.io/pen/debug/eYReNXM/29d1a2e6760a69e567af62c783f2f8de

Figma mockup for the disabled option (to prevent linking twice): https://www.figma.com/file/f8zfcbUP4bpqqs7MB2IsqQ/New-Artifact-Links?node-id=34%3A599
Codepen live mockup for the "already linked" dropdown options (to show ellipsis of title): https://codepen.io/enalean/pen/bGLwwXx/45aea0c272a0fc15eb3ab045a65f5785

Empty
Empty
Status
Trackers
Done
Development
  • [ ] Does it involves User Interface? 
  • [ ] Are there any mockups?
  • [ ] Are permissions checked?
  • [ ] Does it need Javascript development?
  • [ ] Does it need a forge upgrade bucket?
  • [ ] Does it need to execute things in system events?
  • [ ] Does it impact project creation (templates)?
  • [ ] Is it exploratory?
Empty
Details
#24969
Manuel Vacelet (vaceletm)
2022-06-16 13:09
2022-01-04 17:38
3603

References

Follow-ups

User avatar
Joris MASSON (jmasson)2022-05-12 12:48

Add links to specific mockups for the disabled options (already linked).


  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
Joris MASSON (jmasson)2022-05-12 12:00

Edit Acceptance criteria to explain that linking twice the same artifact is not allowed.


  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
Thomas Gorka (tgorka)2022-05-10 11:50

gerrit #25884 integrated into Tuleap 13.8.99.61


  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
  • Category set to Trackers
User avatar
Joris MASSON (jmasson)2022-03-31 12:16

Fix links


  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
  • Attachments image.png removed
User avatar
Joris MASSON (jmasson)2022-03-28 14:34
  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
  • I want to
    -[modal] add and remove links without types 
    +[modal] add and remove links (replace current artifact link) 
User avatar
  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
Thomas Gorka (tgorka)2022-01-28 10:09

gerrit #25003 integrated into Tuleap 13.4.99.156


  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes
User avatar
  • Acceptance criteria
    Something went wrong, the follow up content couldn't be loaded
    Only formatting have been changed, you should switch to markup to see the changes