-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
chore: also export DevToolbarApp
type for API
#9273
Conversation
🦋 Changeset detectedLatest commit: 7518449 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can alias the existing types with the method I commented here!
I think the best thing to do would be to rename the existing types to match the new "Toolbar" name and then add exported aliases for the old names. Does that make sense?
Co-authored-by: Nate Moore <7118177+natemoo-re@users.noreply.github.com>
032adb0
to
371c4a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Changes
interface DevOverlayPlugin
interface DevToolbarApp
Testing
Docs
typed
example in the Dev Toolbar App API reference pagecc @sarah11918