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

[CodeEditor] Explicitly call the latest props.onChange #176296

Merged
merged 2 commits into from
Feb 7, 2024

Conversation

Dosant
Copy link
Contributor

@Dosant Dosant commented Feb 6, 2024

Summary

fix #175684

Refer to the issue #175684 for the problem and reproduction

I tested that this fixes the issue using the story from the reproduction #175670 and the URL drilldown's placeholder issue explained in #175684

…de-editor-stale

# Conflicts:
#	packages/shared-ux/code_editor/impl/code_editor.tsx
@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) labels Feb 6, 2024
@Dosant Dosant requested a review from eokoneyo February 6, 2024 15:00
@Dosant Dosant marked this pull request as ready for review February 6, 2024 15:00
@Dosant Dosant requested a review from a team as a code owner February 6, 2024 15:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@Dosant
Copy link
Contributor Author

Dosant commented Feb 6, 2024

/ci

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #85 / aiops log rate analysis with 'artificial_logs_with_dip_textfield' artificial logs with dip and text field and no zero docs fallback displays index details

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@eokoneyo eokoneyo left a comment

Choose a reason for hiding this comment

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

Verified the stale closure issue is resolved using the reproduction example here #175670 ... Thanks for this fix ❤️

@Dosant Dosant merged commit 4f3bc35 into elastic:main Feb 7, 2024
23 checks passed
@Dosant Dosant self-assigned this Feb 7, 2024
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Feb 7, 2024
fkanout pushed a commit to fkanout/kibana that referenced this pull request Feb 7, 2024
)

## Summary

fix elastic#175684

Refer to the issue elastic#175684 for
the problem and reproduction


I tested that this fixes the issue using the story from the reproduction
elastic#175670 and the URL drilldown's
placeholder issue explained in
elastic#175684
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
)

## Summary

fix elastic#175684

Refer to the issue elastic#175684 for
the problem and reproduction


I tested that this fixes the issue using the story from the reproduction
elastic#175670 and the URL drilldown's
placeholder issue explained in
elastic#175684
fkanout pushed a commit to fkanout/kibana that referenced this pull request Mar 4, 2024
)

## Summary

fix elastic#175684

Refer to the issue elastic#175684 for
the problem and reproduction


I tested that this fixes the issue using the story from the reproduction
elastic#175670 and the URL drilldown's
placeholder issue explained in
elastic#175684
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CodeEditor] Potential bugs due to stale onChange callback
5 participants