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

[Discover] Fix console Range Error #48608

Merged

Conversation

kertal
Copy link
Member

@kertal kertal commented Oct 18, 2019

Summary

Fix multiple RangeErrors in console, when unfolding a list entry:

image

Note: The reason for this error, some changes in Indexpattern, and a reactDirective using it and watchDepth not set to reference.

Fixes: #48586

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] 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 accessibility

For maintainers

- [ ] This was checked for breaking API changes and was labeled appropriately

@kertal kertal self-assigned this Oct 18, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@kertal kertal added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Oct 18, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@kertal kertal added Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes v8.0.0 v7.5.0 v7.6.0 labels Oct 18, 2019
@kertal kertal requested review from a team and LeeDr October 18, 2019 08:14
@nreese
Copy link
Contributor

nreese commented Oct 18, 2019

Is this still a problem? On Master, the problem has been resolved by #47933

Update: looks like the errors are still present. Sorry for the noise

@kertal
Copy link
Member Author

kertal commented Oct 18, 2019

You could review it , then it would be gone :)

@nreese nreese self-requested a review October 18, 2019 14:54
@nreese
Copy link
Contributor

nreese commented Oct 18, 2019

You could review it , them would be gone :)

looking at it now

Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

lgtm
code review, tested in chrome

@kertal
Copy link
Member Author

kertal commented Oct 18, 2019

thanx @nreese, sorry for being pushy

@kertal kertal merged commit 0731d6e into elastic:master Oct 18, 2019
kertal added a commit to kertal/kibana that referenced this pull request Oct 18, 2019
kertal added a commit to kertal/kibana that referenced this pull request Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.5.0 v7.6.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discover RangeError: Maximum call stack size exceeded at SubjectSubscription.toString (<anonymous>)
3 participants