Skip to content

[ResponseOps][Reporting] Allow users to edit scheduled exports#241928

Merged
js-jankisalvi merged 26 commits intoelastic:mainfrom
adcoelho:fe-update-scheduled-reports
Nov 19, 2025
Merged

[ResponseOps][Reporting] Allow users to edit scheduled exports#241928
js-jankisalvi merged 26 commits intoelastic:mainfrom
adcoelho:fe-update-scheduled-reports

Conversation

@adcoelho
Copy link
Contributor

@adcoelho adcoelho commented Nov 5, 2025

Summary

Resolves https://github.com/elastic/kibana-team/issues/2083

This PR allows user to edit scheduled exports.

edit.schedule.mov

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

Release note

Allow users to edit schedule config.

@adcoelho adcoelho self-assigned this Nov 5, 2025
@js-jankisalvi js-jankisalvi self-assigned this Nov 14, 2025
@js-jankisalvi js-jankisalvi changed the title [ResponseOps][Reporting] UI - update scheduled reports Nov 14, 2025
@js-jankisalvi js-jankisalvi changed the title [ResponseOps][Reporting]Allow users to edit scheduled exports Nov 14, 2025
@js-jankisalvi js-jankisalvi added Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.3.0 release_note:enhancement backport:version Backport to applied version labels labels Nov 14, 2025
@js-jankisalvi js-jankisalvi marked this pull request as ready for review November 14, 2025 15:19
@js-jankisalvi js-jankisalvi requested a review from a team as a code owner November 14, 2025 15:19
@elasticmachine
Copy link
Contributor

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

Copy link
Member

@cnasikas cnasikas left a comment

Choose a reason for hiding this comment

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

LGTM! I found a small bug: when I log in without schedule reporting privileges, I need to press the button twice to schedule the export: once to enter the email in the input, and once to actually save the schedule. We may need to show a loading spinner until the user profile is fetched, and then render the input or the form.

Screen.Recording.2025-11-18.at.5.59.35.PM.mov
@cnasikas
Copy link
Member

I tested and the bug is fixed!

@js-jankisalvi js-jankisalvi enabled auto-merge (squash) November 19, 2025 09:58
@js-jankisalvi js-jankisalvi merged commit 93e1386 into elastic:main Nov 19, 2025
12 checks passed
@kibanamachine kibanamachine added backport:skip This PR does not require backporting and removed backport:version Backport to applied version labels labels Nov 19, 2025
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
reporting 231 235 +4

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/reporting-public 113 114 +1

Async chunks

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

id before after diff
reporting 125.8KB 136.8KB +11.0KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
reporting 41.4KB 41.4KB +2.0B
Unknown metric groups

API count

id before after diff
@kbn/reporting-public 120 121 +1

async chunk count

id before after diff
reporting 12 11 -1

History

cc @adcoelho @js-jankisalvi

eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…ic#241928)

## Summary

Resolves elastic/kibana-team#2083

This PR allows user to edit scheduled exports. 


https://github.com/user-attachments/assets/6a136545-5533-4a9d-832a-42dd1f39cede


### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

### Release note

Allow users to edit schedule config.

---------

Co-authored-by: Janki Salvi <jankigaurav.salvi@elastic.co>
Co-authored-by: Janki Salvi <117571355+js-jankisalvi@users.noreply.github.com>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
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 release_note:enhancement Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.3.0

5 participants