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

Revert "Fix typing of default tooltip formatter (#2924)" #3078

Closed

Conversation

furrycatherder
Copy link

This reverts commit da6b9e2.

See #3008 for more information. This change does not appear to be working as intended. This sums it up best (emphasis added):

Moreover, the goal of a formatter is to provide a different representation for ValueType.

@furrycatherder furrycatherder marked this pull request as draft December 4, 2022 22:02
@furrycatherder furrycatherder marked this pull request as ready for review December 4, 2022 22:02
@furrycatherder
Copy link
Author

@arcthur @cravend

@ckifer ckifer added the typescript PRs or Issues surrounding Types or TS refactoring label Dec 13, 2022
@ckifer
Copy link
Member

ckifer commented Dec 29, 2022

@furrycatherder this will break the build. I'm down to fix this issue but we'll need to do it in a way that isn't strictly reverting this commit (and continues to give some type safety)

@ckifer ckifer closed this Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typescript PRs or Issues surrounding Types or TS refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants