Skip to content

Conversation

@KazimirPodolski
Copy link
Contributor

πŸ”— 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

Re-exports useAppConfig for those of us who is not fond of auto-import magic.

πŸ“ Checklist

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 11, 2025

npm i https://pkg.pr.new/@nuxt/ui@5658

commit: 14f0f69

Copy link
Member

@benjamincanac benjamincanac left a comment

Choose a reason for hiding this comment

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

I don't think we should expose useAppConfig from @nuxt/ui export, we should instead add "./vue/composables" and "./vue/composables/*" exports inside package.json.

@KazimirPodolski
Copy link
Contributor Author

I don't think we should expose useAppConfig from @nuxt/ui export, we should instead add "./vue/composables" and "./vue/composables/*" exports inside package.json.

I tried that initially but it wasn't exported properly during my local testing... I'll try again, probably I made a mistake somewhere.

@benjamincanac
Copy link
Member

I'm not even sure this export is needed actually, you should be able to import it like this: import { useAppConfig } from '@nuxt/ui/runtime/vue/composables/useAppConfig.js'

@KazimirPodolski
Copy link
Contributor Author

It works! Hovewer Webstorm is unable to do it automatically:
image

Copy link
Member

Your screenshot made me realize the changes I just made in #5213 will break this import 😬

@benjamincanac
Copy link
Member

@KazimirPodolski Do you have an error when importing from @nuxt/ui/runtime/vue/composables/useAppConfig.js?

@KazimirPodolski
Copy link
Contributor Author

In Webstorm - no, in runtime - gonna check later today, I have some heavy changes ongoing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v4 #4488

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants