Skip to content

feat(frontend): display requests metric#45

Merged
mcollina merged 3 commits intoplatformatic:mainfrom
rozzilla:feat/frontend/display-requests-metric
Mar 26, 2025
Merged

feat(frontend): display requests metric#45
mcollina merged 3 commits intoplatformatic:mainfrom
rozzilla:feat/frontend/display-requests-metric

Conversation

@rozzilla
Copy link
Copy Markdown
Contributor

@rozzilla rozzilla commented Mar 22, 2025

To be rebased once this is merged

Add supports to the requests count metric

@rozzilla rozzilla force-pushed the feat/frontend/display-requests-metric branch from 0982b9f to b7480be Compare March 24, 2025 11:02
@rozzilla rozzilla closed this Mar 24, 2025
@rozzilla rozzilla reopened this Mar 24, 2025
Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>

update

Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>

cleanup

Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>
@rozzilla rozzilla force-pushed the feat/frontend/display-requests-metric branch from d509acb to 1d773fa Compare March 24, 2025 11:04
Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina
Copy link
Copy Markdown
Member

CI is failing

Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>
@rozzilla
Copy link
Copy Markdown
Contributor Author

CI is failing

@mcollina it's passing now

@rozzilla
Copy link
Copy Markdown
Contributor Author

anyway, I worked on this change since we started to have flakiness issues

@mcollina mcollina merged commit c660e74 into platformatic:main Mar 26, 2025
1 check passed
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.

2 participants