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

Adding Pendo integration. #189

Open
wants to merge 1 commit into
base: 8.x-4.x
Choose a base branch
from
Open

Adding Pendo integration. #189

wants to merge 1 commit into from

Conversation

grasmash
Copy link

This is a quick and dirty PR to illustrate how to integrate Pendo with DF.

I intend to publish the Pendo module to Drupal.org as a generic module. The Acquia Pendo module should be kept private.

@grasmash
Copy link
Author

@rlnorthcutt @japerry May be interested in this.

@japerry
Copy link

japerry commented Nov 11, 2019

Yup, in general we shouldn't be bringing the code in like this. And it looks like you've already posted it to https://www.drupal.org/project/pendo which is great. Can I get access to that project?

@grasmash
Copy link
Author

@japerry sure. I've made you a maintainer.

@saltednut
Copy link
Contributor

If you'd like to redo this one as a PR against the composer file we'll slurp it in with all the other acquia_ stuff

@grasmash
Copy link
Author

I can redo the pull request so that the Pendo module is downloaded from Drupal.org. However, the acquia_pendo module should remain private. Where should that be committed?

@japerry
Copy link

japerry commented Nov 11, 2019

@grasmash why should acquia_pendo remain private? It seems to be providing pendo integration for Drupal sites hosted on the Acquia platform. It seems like this too would be good to be available for not just our products using Drupal+Pendo but any of our customers as well.

Maybe have a subfolder inside the pendo module that offers up integrations? Pantheon customers would want to have their own customization when pendo is installed on their platform. And maybe as an alternative to that, we could just integrate the preprocess as an event subscriber so it'd just automatically work without needing to enable extra modules.

@saltednut
Copy link
Contributor

We have a whole library of private stuff I'll dm you Matt

@saltednut
Copy link
Contributor

Is this still an initiative?

@rlnorthcutt
Copy link
Contributor

Just to touch base on this again, acquia/df is the open source version of Demo Framework, so it should only include open source code.

On the other hand, acquia/demo_framework is our proprietary version of DF which allows us to add things like API keys, etc.

So, what do we need to be secret, and why? If we want to offer DF as an option for free tier with Pendo built in, then that would need to be open source anyway - they will have access to the codebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants