You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looks like this was intentionally merged without a typedef, but I'd personally prefer having this typed as any - alternative being littering the code with @ts-ignore for the time being. Generally we try to avoid @ts-ignore over here just so that Typescript will start yelling at us when types are eventually added.
If there's some clean way for me to cast it to any that I'm not thinking about, that would be fine too - the discussion in the PR makes me think I may be missing something easy. Very conscious that opening this issue may have me looking like a bozo - don't hesitate to let me know if I am!
Regardless, it's not a huge deal, just thought this type error was a local problem at first so I figured I'd open the issue for others who might run into it. I won't be offended if you guys want to stick with the original plan and close this issue quickly!
The text was updated successfully, but these errors were encountered:
#920 - PR where StripeResource was exposed.
Looks like this was intentionally merged without a typedef, but I'd personally prefer having this typed as
any
- alternative being littering the code with@ts-ignore
for the time being. Generally we try to avoid@ts-ignore
over here just so that Typescript will start yelling at us when types are eventually added.If there's some clean way for me to cast it to
any
that I'm not thinking about, that would be fine too - the discussion in the PR makes me think I may be missing something easy. Very conscious that opening this issue may have me looking like a bozo - don't hesitate to let me know if I am!Regardless, it's not a huge deal, just thought this type error was a local problem at first so I figured I'd open the issue for others who might run into it. I won't be offended if you guys want to stick with the original plan and close this issue quickly!
The text was updated successfully, but these errors were encountered: