-
Notifications
You must be signed in to change notification settings - Fork 917
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
Remove banner when editing maps visualization #1848
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1848 +/- ##
=======================================
Coverage 67.50% 67.50%
=======================================
Files 3074 3073 -1
Lines 59127 59118 -9
Branches 8988 8986 -2
=======================================
- Hits 39913 39910 -3
+ Misses 17030 17023 -7
- Partials 2184 2185 +1
Continue to review full report at Codecov.
|
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
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'm seeing some internal ips in the commit messages. Could we clean that up?
opensearch-project#933 Signed-off-by: Bandini Bhopi <[email protected]>
79e6fac
to
d083342
Compare
Issue resolved: #933 Signed-off-by: Bandini Bhopi <[email protected]> (cherry picked from commit 53e98a6)
…t#1848) Issue resolved: opensearch-project#933 Signed-off-by: Bandini Bhopi <[email protected]>
Issue resolved: #933 Signed-off-by: Bandini Bhopi <[email protected]> (cherry picked from commit 53e98a6) Co-authored-by: Bandini <[email protected]>
…t#1848) Issue resolved: opensearch-project#933 Signed-off-by: Bandini Bhopi <[email protected]>
Signed-off-by: Bandini Bhopi [email protected]
Description
Fixes #933
Issues Resolved
#933
Check List
yarn test:jest
yarn test:jest_integration
yarn test:ftr