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

examples: update with-plausible README #73305

Merged
merged 2 commits into from
Dec 10, 2024
Merged

Conversation

JamBalaya56562
Copy link
Contributor

@JamBalaya56562 JamBalaya56562 commented Nov 28, 2024

Description

Follow up #73255.
A custom _app.js is no more used to wrap our app with the PlausibleProvider.
Then delete it and convert Header.js to Header.tsx alike other files.

CC: @samcx

Adding or Updating Examples

@ijjk ijjk added the examples Issue was opened via the examples template. label Nov 28, 2024
@ijjk
Copy link
Member

ijjk commented Nov 28, 2024

Allow CI Workflow Run

  • approve CI run for commit: 38e841d

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@JamBalaya56562 JamBalaya56562 force-pushed the plausible branch 4 times, most recently from 6a7db66 to f2aac9a Compare December 9, 2024 01:32
Copy link
Member

@samcx samcx left a comment

Choose a reason for hiding this comment

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

Thank you for submitting a PR!

:lgtm:

@samcx samcx enabled auto-merge (squash) December 10, 2024 06:15
@samcx samcx merged commit 15d1133 into vercel:canary Dec 10, 2024
39 checks passed
@JamBalaya56562 JamBalaya56562 deleted the plausible branch December 10, 2024 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Issue was opened via the examples template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants