Cleanup typescript usage

supabase-community / nuxt-supabase

A supa simple wrapper around Supabase.js to enable usage within Nuxt.

https://supabase.io

MIT License

167 stars 17 forks source link

Cleanup typescript usage #9

Closed sduduzog closed 3 years ago

sduduzog commented 3 years ago

What kind of change does this PR introduce?

Fixes missing type from the vue instance exposed to nuxt
Adds details into setting up the module for TypeScript usage

What is the current behavior?

When using the options api, $supabase is not picked up by typescript
https://github.com/supabase/nuxt-supabase/issues/8

What is the new behavior?

jannikbuscha commented 3 years ago

I get this message when i use the property-decorator. But it worked. Do you know how i can remove this message?

scottrobertson commented 3 years ago

I get this message when i use the property-decorator. But it worked. Do you know how i can remove this message?

Are you running on this branch?

jannikbuscha commented 3 years ago

I get this message when i use the property-decorator. But it worked. Do you know how i can remove this message?

Are you running on this branch?

yes

scottrobertson commented 3 years ago

Thanks @jannikbuscha

@sduduzog this ready to go? What changes need to be done to resolve ^?

sduduzog commented 3 years ago

Thanks @jannikbuscha

@sduduzog this ready to go? What changes need to be done to resolve ^?

Probably some research needs to be done in how 'vue-class-component' works and how types work there. I hope it's as simple as the changes done for nuxt... I'll look into it when I get the chance...

But yes, this PR specifically is ready to go.

@jannikbuscha please could you create a seperate issue for the types issue with class components