Practice manager - Looking for feedback

[bretto36]

Hi All

We use the Xero API but also the Xero Practice Manager API, we've recently had to move to OAUTH2 and am hoping to use the calcinai package so i don't have to reinvent the wheel.

Is it possible to get some feedback on this so i don't go too far down the rabbit hole doing it the wrong way.

[bretto36]

@calcinai Hi! Thank you for the great package. Hoping i could get some of your time to review a Pull Request and tell me if i'm on the right track to add Practice Management support

[calcinai]

@bretto36 looking great! Are you using this in your application currently?

[bretto36]

@calcinai - Yes currently using it in a staging environment. About to go live with it. It's not the full API, but it does handle the features we needed to bring across now that the switch to OAUTH2.0 is on.

These are some of the ways we are using it.

// Invoices
$query = $this->getApi($account)->load(\XeroPHP\Models\PracticeManager\Invoice::class)
                      ->setParameter('detailed', 'true')
                      ->setParameter('from', $dateFrom)
                      ->setParameter('to', $dateTo);

/** @var \XeroPHP\Models\PracticeManager\Invoice[] $invoices */
$invoices = $query->execute();

// Custom Fields
/** @var \XeroPHP\Models\PracticeManager\CustomField[] $customFields */
        $customFields = $this->getApi($account)->load(\XeroPHP\Models\PracticeManager\CustomField::class)->execute();

// Clients
$query = $this->getApi($account)->load(\XeroPHP\Models\PracticeManager\Client::class);

if (!$force) {
    $query->setParameter('modifiedsince', $now->subDays(7)->format('Y-m-d\TH:i:s'));
}

/** @var \XeroPHP\Models\PracticeManager\Client[] $clients */
$clients = $query->execute();

// Loop through clients and can get the sub fields
foreach ($clients as $client) {
    foreach ($client->getCustomFieldValues() as $customFieldValue) {
        // DO something here
    }
}

[bretto36]

Just thought i'd update here. Have been using this in production for over 18 months now. Working well.

[calcinai]

If you fix this exception text I'd be happy to merge!

[bretto36]

@calcinai Have updated the message. Thanks for spotting it

[calcinai]

Nice work @bretto36

[bretto36]

@calcinai Thanks mate. Appreciate the work you've done on the package so far. Come in very handy for me personally. Especially when Xero retired the oauth1a.