Skip to content
This repository has been archived by the owner on Nov 29, 2021. It is now read-only.

Fix active config option #24

Merged
merged 2 commits into from
Jan 14, 2019
Merged

Conversation

jcalonso
Copy link
Contributor

@jcalonso jcalonso commented Dec 21, 2018

The config name for activating the package was incorrectly named when doing the config merge, therefore it was not possible to disable.
Also fixes #23

@jcalonso
Copy link
Contributor Author

jcalonso commented Jan 9, 2019

Fixed merge conflicts

@philkra
Copy link
Owner

philkra commented Jan 14, 2019

thanks for the contribution!

@philkra philkra merged commit b6ad719 into philkra:master Jan 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Elastic still communicate(Send Report) even elastic-apm.active = false
2 participants