search

wearefrank / ladybug-frontend

Frontend for Ladybug
Apache License 2.0
2 stars 7 forks source link

Error not visible to user #379

Closed jacodg closed 4 months ago

jacodg commented 7 months ago

Currently DatabaseStorage doesn't work with PostgreSQL, old GUI shows:

image

But in new GUI everything looks fine:

image

mhdirkse commented 7 months ago

Treedt dit alleen in PoesgreSQL op? Hoe komt het dat er een groot object wordt opgeslagen in de LADYBUG tabel? Hoe groot moet het object zijn om deze fout te triggeren?

jacodg commented 7 months ago

Tot nu toe alleen bij PostgreSQL gezien. We slaan de reports op in een BLOB. Ook bij kleine reports van zo'n 2500 bytes treedt dit op. Ik heb lokaal nu een transactie manager erbij geconfigureerd dat lost het probleem op.

mhdirkse commented 7 months ago

Hoort niet op dit bord. Gekopiëerd naar https://github.com/wearefrank/ladybug-ff-cypress-test/issues/9.

jacodg commented 6 months ago

Waarom hoort dit issue hier niet? Het probleem moet toch verholpen worden in de front-end? Los daarvan moet er ook een test van komen, als je daar liever een issue voor hebt op een ander bord prima.

mhdirkse commented 6 months ago

The issue is that a low-level error is not properly shown in the ladybug bèta user interface. Do you have a suggestion on how to produce a relevant low-level error in the ladybug-test-webapp?

mhdirkse commented 6 months ago

@jacodg I can do this research myself. I shall try to reproduce the issue by updating ladybug-test-webapp. If I can do so, I can then write a Cypress test in ladybug-frontend.

jacodg commented 6 months ago

I think you can create a Storage class in the test webapp that extends nl.nn.testtool.storage.memory.Storage and have getWarningsAndErrors() return an error message. And configure this new class as an extra storage/view in springTestToolTestWebapp.xml

mhdirkse commented 6 months ago

I will write an API endpoint in the Ladybug project that calls getWarningsAndErrors(). When we have that, I will inform Matthijs.

mhdirkse commented 6 months ago

Matthijs heeft mij gevraagd om de Cypress test te maken als hij dit feature geïmplementeerd heeft. Dat zal ik doen.

MatthijsSmets commented 6 months ago

On seconds thought, this is not something that should be shown in the ladybug-frontend. I think it would fit more if this was something that is checked when the backend is started.

jacodg commented 5 months ago

This error will only occur when a message is being processed and not at start-up. The user should be informed of such runtime errors

mhdirkse commented 3 months ago

Wordt story 5020.