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 PAN Analytics add-on to demo, for product analytics #665

Merged
merged 4 commits into from
Oct 31, 2024
Merged

Conversation

tabacitu
Copy link
Member

WHY

BEFORE - What was wrong? What was happening before this PR?

Pedro had created an addon for PAN Analytics but we weren't showing it anywhere.

AFTER - What is happening after this PR?

We are showing it in our Demo, and tracking what people click on our Welcome page and our Menu:

CleanShot 2024-10-30 at 17 08 47@2x

HOW

How did you achieve that, in technical terms?

Install package and add data-pan=name in a few places in our custom blade files.

Is it a breaking change or non-breaking change?

Non-breaking. BUT. The composer.json needs to change once the add-on is released, to load the first major version.

How can we test the before & after?

Todo:

  • Switch to this branch of the demo.
  • Run composer install and php artisan migrate.
  • Then play around in the demo to load up the analytics db with some entries.
  • Then go to Addons > Analytics to see the results.

@tabacitu tabacitu changed the title added PAN Backpack add-on for product analytics Add PAN Analytics add-on to demo, for product analytics Oct 30, 2024
@tabacitu tabacitu mentioned this pull request Oct 30, 2024
7 tasks
@pxpm pxpm merged commit 6b13872 into main Oct 31, 2024
3 checks passed
@pxpm pxpm deleted the add-pan-add-on branch October 31, 2024 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants