-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Reporting/Screenshots] add error for no shared items found on the page #52022
[Reporting/Screenshots] add error for no shared items found on the page #52022
Conversation
This comment has been minimized.
This comment has been minimized.
8d7fa97
to
0951fde
Compare
Pinging @elastic/kibana-app-arch (Team:AppArch) |
Ready for re-review |
@@ -313,7 +313,7 @@ export class VisualizeEmbeddable extends Embeddable<VisualizeInput, VisualizeOut | |||
|
|||
div.setAttribute('data-test-subj', 'visualizationLoader'); | |||
div.setAttribute('data-shared-item', ''); | |||
div.setAttribute('data-shared-item-container', ''); | |||
div.setAttribute('data-shared-items-container', ''); |
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.
Whoops, this is affecting Dashboard reports now, and causing them to break.
💔 Build Failed |
0951fde
to
149022b
Compare
149022b
to
0f474f9
Compare
@elasticmachine merge upstream |
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.
LGTM, reviewed code only
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
…ge (elastic#52022) * [Reporting/Screenshots] add error for no shared items container found on the page * wording adjustment
…ge (elastic#52022) * [Reporting/Screenshots] add error for no shared items container found on the page * wording adjustment
Summary
This PR helps the triaging of #51309 as an issue. If there is a problem with the Kibana page where the Reporting integration will not work, Reporting will log an appropriate error.
This PR follows on #52109
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.[ ] This was checked for cross-browser compatibility, including a check against IE11[ ] Documentation was added for features that require explanation or tutorials[ ] Unit or functional tests were updated or added to match the most common scenarios[ ] This was checked for keyboard-only and screenreader accessibilityFunctional tests to isolate the screen capture workflow is dependent on a draft PR: #48588