Thank you for your contribution. You probably want to take a look at [Rest API Design](https://www.mulesoft.com/resources/api/what-is-rest-api-design).

1. [this line](https://github.com/pcsg/pcsg-makerlog-php-client/commit/47452cb0fe7675baab284a5771dc79d6c7ac0391#diff-f0c722a0daf1037815467c8e1c129611R11) seems duplicate.
2. Your `getProductData` has been called in its many APIs to get the sub-field which may be cached.
3. The `getData` and many other APIs has a long chain in getting its field data, which may throw exception e.g. null pointer. The try-catch will hide the errors and prevent early identification of the risks - which is generally a bad idea.
4. When performing `addUserToTheTeam`, have you thought about multiple requests i.e. race conditions?

Your contribution has been evaluated according to [Utopian policies and guidelines](https://join.utopian.io/guidelines), as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, [click here](https://review.utopian.io/result/3/2-2-2-2-2-2-1-).

---- 
Need help? Chat with us on [Discord](https://discord.gg/uTyJkNm).

[[utopian-moderator]](https://join.utopian.io/)