marshallswain / feathers-pinia

Connect your Feathers API to the elegant data store for Vue
52 stars 22 forks source link

fix: useFind default typing broken #173

Open mrfrase3 opened 1 month ago

mrfrase3 commented 1 month ago

just pulled my hair out comparing the setup from project A to project B, where the data field returned from useFind in project B was not inferring the service type, and instead insisting on unknown[], finally looked at the package versions, long behold project B is running on the latest and A in on 4.4.0 😅

looks like this is the culprit: https://github.com/marshallswain/feathers-pinia/commit/4804ef4aaa842d52c3f21d491bccb5bcf5a33f09

if you don't explicitly provide a type for M, it assumes it's unknown. I managed to fix it by setting a default on the declaration <M = SvcModel>

Also taking out the ambiguity of the type, either its the override or the default type, not both