Get completed items

[jkawamoto]

It's useful if the client can get completed tasks. This PR implements get_completed_items that calls Get completed items API.

Since it looks like models are migrating to use dataclass instead of attrs (#85), this PR also uses dataclass to define models.

Related to #32 and #56.

[theptrk]

This would be very useful!

I hope this library isn't REST API only since the Sync API still has useful endpoints such as activity logs.

[lefcha]

The PR looks great, apart from the one comment I had. If it's addressed we can merge it. Thanks!

[jkawamoto]

Thank you for your review. I removed the docstrings and rebased this branch onto the latest main.

[theptrk]

thanks for merging this 🎉

[brunorosilva]

Great! @lefcha, I'm working on #96 which will also use the Sync API, will this be a problem in the future? The doc seems clear about not using calls to the Sync API.

[lefcha]

@brunorosilva It's OK to use auxiliary API endpoints from the Sync API in this library. What we don't want is to use the sync command itself. Unfortunately the archiving functionality is part of the sync command, so a PR would not be accepted.

You would need to wait for us to implement this in the REST API. CC @PotHix

[brunorosilva]

OK @lefcha , thanks for the info.