-
Notifications
You must be signed in to change notification settings - Fork 194
[Internal][ResponseOps][9.3 and Serverless]: Improvements to Scheduled reports #4545
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
base: main
Are you sure you want to change the base?
[Internal][ResponseOps][9.3 and Serverless]: Improvements to Scheduled reports #4545
Conversation
Vale Linting ResultsSummary: 1 suggestion found 💡 Suggestions (1)
The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
🔍 Preview links for changed docs |
js-jankisalvi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good 👍
natasha-moore-elastic
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a couple of non-blocking comments, LGTM!
explore-analyze/report-and-share/automating-report-generation.md
Outdated
Show resolved
Hide resolved
| - {applies_to}`stack: ga 9.3+` Disabling a schedule allows you to keep a record of it on the **Reporting** page, but permanently turns the schedule off. To restart a disabled schedule, select **Enable schedule**. | ||
| - {applies_to}`stack: ga 9.1-9.2` Disabling a schedule allows you to keep a record of it on the **Reporting** page, but permanently turns the schedule off. To restart the schedule, you must create a new one. | ||
| - {applies_to}`stack: ga 9.3+` **Enable schedule**: Turn a disabled schedule back on. | ||
| - {applies_to}`stack: ga 9.3+` **Edit schedule config**: Modify an existing schedule. | ||
| - {applies_to}`stack: ga 9.3+` **Delete schedule**: Permanently stop a schedule and remove the schedule's record from the **Reporting** page. You can't recover a deleted schedule. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like the number of labels we need to use here could be visually overwhelming. This might be a good use case for applies-switch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went back and forth on this too. I'll try the applies to switch and see if it's less visually overwhelming.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went back and forth on this too. I'll try the applies to switch and see if it's less visually overwhelming.
| - **Disable schedule**: {applies_to}`stack: ga 9.1+` Disabling a schedule allows you to keep a record of it on the **Reporting** page, but permanently turns the schedule off. To restart the schedule, you must create a new one. | ||
| - **Delete schedule**: {applies_to}`stack: ga 9.3+` Deleting a schedule permanently stops it and removes the schedule's record from the **Reporting** page. You can't recover a deleted schedule. | ||
| To manage a scheduled report, you can take the following actions from the **Schedules** tab on the **Reporting** page: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sentence refers to managing "a scheduled report" but the listed actions are about managing schedules themselves. Should the intro say "To manage a schedule" instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, good catch!
Co-authored-by: natasha-moore-elastic <137783811+natasha-moore-elastic@users.noreply.github.com>
Summary
Updated docs for scheduled reports as follows:
applies_totags for the "Schedule and share reports" section.This PR fixes the following issues:
Generative AI disclosure