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

[Canvas] XY. Step 1. Remove all specific logic, related to aggType. #108372

Closed
wants to merge 23 commits into from

Conversation

Kuznietsov
Copy link
Contributor

@Kuznietsov Kuznietsov commented Aug 12, 2021

Completes part of #110430 and #101377

Step 1. XY. Remove all specific logic, related to aggType completed.

@Kuznietsov Kuznietsov changed the title [Canvas] XY_VIS. [Canvas] XY. Step 1. Removed all specific logic, related to aggType. Aug 30, 2021
@Kuznietsov Kuznietsov changed the title [Canvas] XY. Step 1. Removed all specific logic, related to aggType. [Canvas] XY. Step 1. Removed all specific logic, related to aggType. Aug 30, 2021
@Kuznietsov Kuznietsov changed the title [Canvas] XY. Step 1. Removed all specific logic, related to aggType. [Canvas] XY. Step 1. Remove all specific logic, related to aggType. Aug 30, 2021
Copy link
Contributor

@nickofthyme nickofthyme left a comment

Choose a reason for hiding this comment

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

Code changes LGTM, I agree with the comments for proposed changes.

@Kuznietsov
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

⏳ Build in-progress, with failures

History

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

3 similar comments
@kibanamachine
Copy link
Contributor

⏳ Build in-progress, with failures

History

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

@kibanamachine
Copy link
Contributor

⏳ Build in-progress, with failures

History

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

@kibanamachine
Copy link
Contributor

⏳ Build in-progress, with failures

History

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

@Kuznietsov Kuznietsov requested review from a team, academo and dasansol92 and removed request for a team, ashokaditya, joeypoon, academo and dasansol92 September 10, 2021 14:25
@Kuznietsov Kuznietsov removed the WIP Work in progress label Sep 10, 2021
@Kuznietsov Kuznietsov changed the title [WIP][Canvas] XY. Step 1. Remove all specific logic, related to aggType. [Canvas] XY. Step 1. Remove all specific logic, related to aggType. Sep 10, 2021
@Kuznietsov Kuznietsov force-pushed the xy_chart branch 2 times, most recently from c7f7f9d to 4fe713e Compare September 10, 2021 14:40
@Kuznietsov
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

⏳ Build in-progress, with failures

History

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

cc @Kunzetsov

@Kuznietsov
Copy link
Contributor Author

@elasticmachine merge upstream

@legrego
Copy link
Member

legrego commented Sep 10, 2021

Hey @Kunzetsov, would you mind continuing this work in a new PR? An earlier push pinged a large number of folks across several teams, which floods notifications until this is either merged or closed 🙏

@Kuznietsov
Copy link
Contributor Author

Kuznietsov commented Sep 10, 2021

Hey @Kunzetsov, would you mind continuing this work in a new PR? An earlier push pinged a large number of folks across several teams, which floods notifications until this is either merged or closed 🙏

@legrego, sure I can. Sorry for that.

@Kuznietsov Kuznietsov closed this Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Canvas impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:x-large Extra Large Level of Effort release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants