picqer / moneybird-php-client

PHP Client for Moneybird V2
MIT License
82 stars 77 forks source link

listVersions for estimates only returning id? #141

Closed vissr closed 6 years ago

vissr commented 6 years ago

Has anyone experienced any trouble with the synchronization api for the estimates model? Suddenly running the following code:

$estimateVersions = $moneybird->estimate()->listVersions();

Results in the following:

schermafbeelding 2018-03-02 om 16 24 15

In short, the only thing returned is the id, but no version. Anyone else experienced trouble with this?

stephangroen commented 6 years ago

It happens because the Estimates entity does not have a version attribute. You can add it to the fillables at https://github.com/picqer/moneybird-php-client/blob/master/src/Picqer/Financials/Moneybird/Entities/Estimate.php#L29 and then it will work. Please submit a PR :)