J-Michalek ยท GitHub

Conversation

@J-Michalek

Copy link Copy Markdown

Contributor

๐Ÿ”— Linked issue

Resolves #4970

โ“ 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

I've updated the import path for the TVConfig type from @nuxt/ui to #ui/types/tv as it was not being resolved properly (hopefully I'm not missing something) and made the AppConfigUI type fallback to any.

After this change the types seem to work as expected in app.config.ts.

๐Ÿ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@J-Michalek

@pkg-pr-new

Copy link Copy Markdown
npm i https://pkg.pr.new/@nuxt/ui@4971

commit: ecf6a43

@vercel

benjamincanac

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

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

@J-Michalek I'd rather export * from './tv' in src/runtime/types/index.ts rather than changing this!

I might have forgot this in some refactor ๐Ÿ˜ฌ

benjamincanac

benjamincanac

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

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

Thanks ๐Ÿ˜Š

@benjamincanac benjamincanac changed the title fix(templates): update import path for TVConfig type fix(types): export missing tv types

Sep 14, 2025

@vercel

@benjamincanac

@J-Michalek

2 participants

@J-Michalek @benjamincanac

Read the original on github.com โ†—