Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@vachmara
Copy link
Contributor

@vachmara vachmara commented Oct 7, 2025

πŸ”— Linked issue

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

This pull request introduces support for Databuddy Analytics by updating type definitions. The changes ensure that Databuddy Analytics can be properly referenced and configured within the application's script registry.

Analytics integration:

  • Added import for DatabuddyAnalyticsInput in src/runtime/types.ts to enable type checking for Databuddy Analytics configuration.
  • Extended the ScriptRegistry interface in src/runtime/types.ts to include an optional databuddyAnalytics property, allowing Databuddy Analytics to be registered alongside other analytics providers.

@vercel
Copy link

vercel bot commented Oct 7, 2025

@vachmara is attempting to deploy a commit to the Nuxt Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 7, 2025

Open in StackBlitz

npm i https://pkg.pr.new/nuxt/scripts/@nuxt/scripts@528

commit: e1735aa

@vachmara vachmara changed the title fix: databuddy type safe script registry fix: databuddy type script registry Oct 7, 2025
@vachmara vachmara changed the title fix: databuddy type script registry fix: databuddy types script registry Oct 7, 2025
@harlan-zw
Copy link
Collaborator

Oh yeh thanks, I missed the in the review

@harlan-zw harlan-zw merged commit f53dc7e into nuxt:main Oct 7, 2025
3 of 5 checks passed
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.

2 participants