Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changed terms of Office Max Rule Level #4508

Merged
merged 1 commit into from
Sep 14, 2022

Conversation

Tostti
Copy link
Member

@Tostti Tostti commented Sep 13, 2022

Description

This PR fixes an issue with the main Office 365 dashboard, in which the Max Rule Level was not showing the correct value.

Changes

  • Modified the file public/components/overview/metrics/metrics.tsx for the office Max Rule Level to use the field rule.level instead of timestamp.

Closes #4355

Screenshots

  • Before the changes:

image

  • After the changes:
    image

Tests

Scenario: Open the main Office 365 dashboard
Given the alerts on the specified timeframe
When the maximum alert level is defined
Then the Max Rule Level should show the correct value and match with the value shown in the Events by severity over time graphic

Copy link

@sempervictus sempervictus left a comment

Choose a reason for hiding this comment

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

Thank you, that looks correct.

Copy link
Member

@AlexRuiz7 AlexRuiz7 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Mayons95
Copy link
Contributor

TR: ✔️
image

LGTM !

@Mayons95 Mayons95 self-requested a review September 14, 2022 12:21
@AlexRuiz7 AlexRuiz7 merged commit 02b581b into 4.4-7.10 Sep 14, 2022
@AlexRuiz7 AlexRuiz7 deleted the fix/office365-max-rule-level branch September 14, 2022 12:27
github-actions bot pushed a commit that referenced this pull request Sep 14, 2022
github-actions bot pushed a commit that referenced this pull request Sep 14, 2022
Tostti added a commit that referenced this pull request Sep 14, 2022
Changed terms of Office Max Rule Level (#4508)

(cherry picked from commit 02b581b)

Co-authored-by: Tostti <42900763+Tostti@users.noreply.github.com>
Tostti added a commit that referenced this pull request Sep 14, 2022
Changed terms of Office Max Rule Level (#4508)

(cherry picked from commit 02b581b)

Co-authored-by: Tostti <42900763+Tostti@users.noreply.github.com>
@Mayons95 Mayons95 mentioned this pull request Dec 7, 2022
1 task
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.

Office365 view presents incorrect value for max event level
4 participants