Delete commissioning packages

[eoye]

Pull requests practices for the SSI team

Aim of the PR

This PR fixes AB#208957.

The goal of this PR is to add functionality for deleting commissioning packages.

Implementation

I added a new sidebar item with a delete icon. When the user clicks the button, a dialog opens. The dialog shows all commissioning packages, and the user must choose which packages they want to delete. The active package will be automatically checked, as it is likely the user wants to delete the package that is currently active. When you delete the last commissioning package, a new initial commissioning package will be created.

Type of change

• Bug fix
• New feature
• Breaking change
• This change requires a documentation update

How Has This Been Tested?

This has been tested by running the demo and seeing that it is possible to delete commissioning packages. See screenshots:

[NoraKri]

Fin og ryddig kode! Ser ut som slettingen nesten er i mål. Kan ta utgangspunkt i funksjonen cleanTripleStore i Triplestore.ts for sletting i db. Ellers er det lurt som du skriver å flytte slettingen ut i en egen komponent.

Et par designtanker rundt slettingen:

• Kanskje aktiv pakke burde bli valgt automatisk? Ser for meg at man ofte vil slette den pakken man er i.
• Fint om fargen på pakken er med i listen over pakker når du skal slette, så blir det lettere å se hvilken pakke som er hvilken i P&IDen.

Veldig bra jobbet! 🎄

[markus-ap]

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

✅ license/snyk check is complete. No issues have been found. (View Details)

✅ code/snyk check is complete. No issues have been found. (View Details)

[eoye]

Fin og ryddig kode! Ser ut som slettingen nesten er i mål. Kan ta utgangspunkt i funksjonen cleanTripleStore i Triplestore.ts for sletting i db. Ellers er det lurt som du skriver å flytte slettingen ut i en egen komponent.

Et par designtanker rundt slettingen:

• Kanskje aktiv pakke burde bli valgt automatisk? Ser for meg at man ofte vil slette den pakken man er i.
• Fint om fargen på pakken er med i listen over pakker når du skal slette, så blir det lettere å se hvilken pakke som er hvilken i P&IDen.

Veldig bra jobbet! 🎄

Takk, nå har jeg fikset at pakkene slettes i triplestoren og flyttet dialogen ut i en egen komponent! Synes begge designtankene dine var gode, så har lagt til farge på pakkene i listen over pakker for sletting, og gjort sånn at den aktive pakken automatisk er sjekket av! 🙌

[daghovland]

Det funker fint på firefox på linux. Jeg har ikke noen gode innspill her, er ikke god nok i react til å fange opp om noe er feil tror jeg.