Skip to content

[ResponseOps][MaintenanceWindow] Remove archive banner from edit MW page#230971

Merged
adcoelho merged 3 commits intoelastic:mainfrom
adcoelho:ui-archive-mw
Aug 12, 2025
Merged

[ResponseOps][MaintenanceWindow] Remove archive banner from edit MW page#230971
adcoelho merged 3 commits intoelastic:mainfrom
adcoelho:ui-archive-mw

Conversation

@adcoelho
Copy link
Contributor

@adcoelho adcoelho commented Aug 7, 2025

Closes #230768

Summary

I removed the banner as discussed offline and we now only display the Archive button.

Screenshot 2025-08-07 at 13 39 36

The modal text was reworded (@nastasha-solomon could you please proofread the copy?).

Screenshot 2025-08-07 at 13 29 41
Something else I noticed was that we have an ugly margin in the edit page, right below the `Filters` section.Screenshot 2025-08-07 at 13 24 04
I was going to remove it, but looking at the code, [this PR](https://github.com//pull/227888) should fix it, so I didn't touch it.
@adcoelho adcoelho self-assigned this Aug 7, 2025
@adcoelho adcoelho requested a review from a team as a code owner August 7, 2025 11:40
@adcoelho adcoelho added enhancement New value added to drive a business result release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// Feature:Maintenance Window labels Aug 7, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@js-jankisalvi js-jankisalvi self-requested a review August 11, 2025 08:30
Copy link
Contributor

@js-jankisalvi js-jankisalvi left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@adcoelho adcoelho enabled auto-merge (squash) August 12, 2025 09:32
@adcoelho adcoelho disabled auto-merge August 12, 2025 09:35
@adcoelho adcoelho enabled auto-merge (squash) August 12, 2025 09:38
@adcoelho adcoelho merged commit 647df81 into elastic:main Aug 12, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
alerting 88.0KB 88.0KB +11.0B

History

cc @adcoelho

NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Aug 18, 2025
…age (elastic#230971)

Closes elastic#230768

## Summary

I removed the banner as discussed offline and we now only display the
`Archive` button.

<img width="1374" height="126" alt="Screenshot 2025-08-07 at 13 39 36"
src="https://github.com/user-attachments/assets/6527fd55-5a4f-4015-bb82-899845ae9b0e"
/>

The modal text was reworded (@nastasha-solomon could you please
proofread the copy?).

<img width="1005" height="409" alt="Screenshot 2025-08-07 at 13 29 41"
src="https://github.com/user-attachments/assets/989e9b19-9f96-4c2f-8757-d13fa45f744b"
/>

<details><summary>Something else I noticed was that we have an ugly
margin in the edit page, right below the `Filters`
section.</summary><img width="1416" height="370" alt="Screenshot
2025-08-07 at 13 24 04"
src="https://github.com/user-attachments/assets/31b7e7c1-d1a4-436e-a121-f56d64ec7598"
/></details>
I was going to remove it, but looking at the code, [this
PR](elastic#227888) should fix it, so I
didn't touch it.
qn895 pushed a commit to qn895/kibana that referenced this pull request Aug 26, 2025
…age (elastic#230971)

Closes elastic#230768

## Summary

I removed the banner as discussed offline and we now only display the
`Archive` button.

<img width="1374" height="126" alt="Screenshot 2025-08-07 at 13 39 36"
src="https://github.com/user-attachments/assets/6527fd55-5a4f-4015-bb82-899845ae9b0e"
/>

The modal text was reworded (@nastasha-solomon could you please
proofread the copy?).

<img width="1005" height="409" alt="Screenshot 2025-08-07 at 13 29 41"
src="https://github.com/user-attachments/assets/989e9b19-9f96-4c2f-8757-d13fa45f744b"
/>

<details><summary>Something else I noticed was that we have an ugly
margin in the edit page, right below the `Filters`
section.</summary><img width="1416" height="370" alt="Screenshot
2025-08-07 at 13 24 04"
src="https://github.com/user-attachments/assets/31b7e7c1-d1a4-436e-a121-f56d64ec7598"
/></details>
I was going to remove it, but looking at the code, [this
PR](elastic#227888) should fix it, so I
didn't touch it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting enhancement New value added to drive a business result Feature:Maintenance Window release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.2.0

4 participants