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

Add Flexmonster Pivot Table to known plugins list #39680

Closed
wants to merge 2 commits into from
Closed

Add Flexmonster Pivot Table to known plugins list #39680

wants to merge 2 commits into from

Conversation

flexmonster
Copy link

Summary

Added information about a pivot table plugin to the list of known plugins.

@elasticmachine
Copy link
Contributor

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@mattkime
Copy link
Contributor

@flexmonster could you agree to the CLA? https://www.elastic.co/contributor-agreement

@mattkime mattkime added release_note:skip Skip the PR/issue when compiling release notes v8.0.0 labels Jun 26, 2019
@mattkime
Copy link
Contributor

retest

@mattkime
Copy link
Contributor

mattkime commented Jun 26, 2019

I think it would be better to make this PR against master. We're only making bug fixes on the 7.2 branch.

@mattkime mattkime self-assigned this Jun 26, 2019
@flexmonster
Copy link
Author

flexmonster commented Jun 26, 2019

Hi Matthew,

Thank you for your time and consideration!

It would be great to add information about the plugin to the master branch.

Also, we would like to note that we opened an issue with the same purpose, therefore, this pull request and the issue can be considered equivalent.

Grateful for your help!

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@mattkime
Copy link
Contributor

@flexmonster I'd be happy to help you with a PR against master.

@mattkime mattkime mentioned this pull request Jun 26, 2019
7 tasks
@flexmonster
Copy link
Author

@mattkime
Hi Matthew!

Is it necessary to sign the CLA to add our plugin to the list?

Regards,
Flexmonster Team

@nickpeihl
Copy link
Member

A new PR was opened and merged against the master branch. As such, I am going to close this. #41655

@nickpeihl nickpeihl closed this Jul 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants