Include shared vehicles in getAccountVehicles

samrum / OnStarJS

NodeJS Library for making OnStar API requests

MIT License

85 stars 17 forks source link

Closed FieryFlames closed 5 months ago

FieryFlames commented 7 months ago

Fixes #204. I also removed a random space from includeEntitlements. This should be tested on a account with no shared vehicles.

samrum commented 5 months ago

@BigThunderSR could you test this change?

BigThunderSR commented 5 months ago

@BigThunderSR could you test this change?

Will take a look and see.

BigThunderSR commented 5 months ago

@samrum, this does appear to be working.

Tested with my account which is primary and has both vehicles - works.
Tested with my wife's account which I have shared both vehicles to - works. (Note: This did not work prior to this code change)

samrum commented 5 months ago

Thanks @BigThunderSR and @FieryFlames !