search

wearefrank / ladybug-frontend

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

Show message metadata on 'view report page' #458

Open mhdirkse opened 2 months ago

mhdirkse commented 2 months ago

When you open a report and click on the root node, the value of that node appears. Presently, only the XML of the report is shown. Niels Meijer asked whether along with that XML, report metadata can be shown. He suggested to put the metadata under the XML. The metadata may also be hidden so that it is only shown if you click somewhere. The most important item in this value window remains the XML of the report; the metadata is a nice addition.

mhdirkse commented 2 months ago

@jacodg appte mij dat hij dit een goed issue vindt.

mhdirkse commented 2 months ago

Ik heb de button over het hoofd gezien om metadata te tonen:

image

Blijft de suggestie van Niels over om de metadata onder de XML te hebben in plaats van erboven. Is dit een goed idee?

MatthijsSmets commented 2 months ago

I think it makes more sense to have the metadata table at the top, like it is now, I'll close this because this is a styling suggestion and not an issue.

nielsm5 commented 2 months ago

That's the ladybug's metadata, which is not what I was talking about. We have over 30 keys we would like to display.

nielsm5 commented 2 months ago

Overigens, mijn doel is het inzichtelijk maken van de metadata, (niet ladybug metadata!!!). Waar het komt te staan maakt mij niet uit.

jacodg commented 2 months ago

See also: https://github.com/frankframework/frankframework/issues/4208 https://github.com/frankframework/frankframework/issues/3448 https://github.com/frankframework/frankframework/issues/5163

mhdirkse commented 2 months ago

Ik denk dat dit niet alleen een frontend issue is. De metadata moet ook doorgegeven worden door de backend. Misschien moet ook het FF! worden aangepast om de metadata door te geven aan Ladybug's Testtool class.

nielsm5 commented 2 months ago

Ik denk dat dit niet alleen een frontend issue is. De metadata moet ook doorgegeven worden door de backend. Misschien moet ook het FF! worden aangepast om de metadata door te geven aan Ladybug's Testtool class.

Dat klopt zeker. Het is nieuwe functionaliteit.

nielsm5 commented 3 weeks ago

See also: https://github.com/frankframework/frankframework/issues/4478

jacodg commented 3 weeks ago

@MatthijsSmets for which question do you need response?