Added count to ModeledResponse

[DanielW093]

What kind of change does this PR introduce?

Added count to ModeledResponse, with ability to specify what kind of count should be retrieved

[grdsdev]

Hey @DanielW093, thanks for adding that! Can you add a test case for that change?

[DanielW093]

Hey @DanielW093, thanks for adding that! Can you add a test case for that change?

Of course, happy to :)

[grdsdev]

@DanielW093, I’ve been thinking, instead of adding a new method, why don’t we just add a count property to the ModeledResponse? That way, it’ll be more in line with postgrest-js, as shown in: https://github.com/supabase/postgrest-js/blob/master/src/types.ts#L17

[DanielW093]

@DanielW093, I’ve been thinking, instead of adding a new method, why don’t we just add a count property to the ModeledResponse? That way, it’ll be more in line with postgrest-js, as shown in: https://github.com/supabase/postgrest-js/blob/master/src/types.ts#L17

If you think that makes more sense then I'll defer to your judgement! I'll admit I didn't dig too deeply into the codebase before making this change, so if that's better and more consistent with other libraries then that's probably the right approach

[grdsdev]

Sure, so let's go with the count property then

[DanielW093]

@grdsdev I've implemented my version of the suggested change and added tests, it all appears to be working as expected in my project

[grdsdev]

Thanks @DanielW093