Skip to content

Add host and application to a report#833

Open
mhdirkse wants to merge 32 commits into
masterfrom
foks
Open

Add host and application to a report#833
mhdirkse wants to merge 32 commits into
masterfrom
foks

Conversation

@mhdirkse

@mhdirkse mhdirkse commented Jun 4, 2026

Copy link
Copy Markdown
Member

No description provided.

@mhdirkse mhdirkse marked this pull request as ready for review June 5, 2026 13:14
@mhdirkse mhdirkse requested a review from jacodg June 5, 2026 13:14
@mhdirkse

mhdirkse commented Jun 5, 2026

Copy link
Copy Markdown
Member Author

De debug table will look like this:

image

@@ -0,0 +1,44 @@
describe('Tests about host and application', () => {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not call this file foks.

@mhdirkse mhdirkse marked this pull request as draft June 8, 2026 09:01
@mhdirkse

mhdirkse commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

I tested the updated default XSLT transformation in an online XSLT tester. It does not produce errors and it puts IGNORED as the value of the Host attribute.

@mhdirkse

mhdirkse commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

The last commit introduces a Spring profile in ladybug-test-webapp that both uses database storages and works with host and application. I checked with that profile that the host and the application appear in the debug table. The host and the application also appear in the metadata table of the report node. This is the case both in the debug tab and in a separate tab.

@mhdirkse mhdirkse marked this pull request as ready for review June 11, 2026 08:52
@mhdirkse

Copy link
Copy Markdown
Member Author

See https://github.com/wearefrank/ladybug/actions/runs/27629213132/job/81699253730 for tests in combination with FF!.

@mhdirkse

Copy link
Copy Markdown
Member Author

See https://github.com/wearefrank/ladybug/actions/runs/27629213132/job/81699253730 for tests in combination with FF!.

Test succeeds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant