search

wearefrank / ladybug-frontend

Frontend for Ladybug
Apache License 2.0
1 stars 6 forks source link

Add support for dynamic copying #196

Open celmoussaoui opened 2 years ago

celmoussaoui commented 2 years ago

Copying a report from the display of debug now always happens into testStorage, this should be dynamic instead (for now we hardcode it to Test)

mhdirkse commented 2 months ago

Ik denk niet dat dit nog een requirement is. Ik stel voor dat we hem sluiten en niet opnemen in de user stories. Wat vind jij, @jacodg ?

jacodg commented 2 months ago

Reports kunnen kopiëren tussen storages lijkt me wel nodig, bijvoorbeeld als je in de Debug tab een report ziet die je graag wil bewaren als unit test in de test tab voor regressie testen. Op zich kan dit nu al met de "for now we hardcode it to Test", daarmee dekken we het meeste denk ik wel af. Maar het zou kunnen dat je meerdere views met meerdere storages hebt geconfigureerd, dus zou het mooi zijn als er dynamisch wordt gekeken welke storages er allemaal zijn en de gebruiker daar tussen kan kiezen ipv dat er altijd een hardcoded kopie wordt gedaan naar de test storage (wat feitelijk ook maar een bepaalde configuratie is, al is dat wel de default configuratie)

mhdirkse commented 1 month ago

@jacodg, als je dit wilt - zijn er dan ook meerdere tabs in plaats van één test tab? Of wil je dat je in de test tab de storage kunt kiezen? Ik vind het juist zo duidelijk zonder dat we dit issue meenemen - rapporten in de debug tab zijn originele captures en in de test tab staan rapporten die geëdit zijn tot regressietesten.

mhdirkse commented 2 weeks ago

Dit issue gaat erom dat er nu één tab Test is en dat er misschien meer zouden moeten zijn. De testen verdelen over meerdere tabs. Als we dit gaan doen, dan moeten we de user stories updaten zodat duidelijk wordt hoe dit past bij de bestaande functionaliteit.