New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ConnectWise API Base Path Change #9
Comments
SolutionsThese are my suggested solutions in order of preference.
No matter which solution, this module/lib should provide a function or method (a part of |
@sgtoj I am a fan of option 1 + 3 or 4. @mattheyan what do you think about this? |
There is a problem that will need to resolve first. The function the builds the CW Manage endpoint, will need to be async/promise or calls a callback because it's a web request. This complicates the logic little more than I originally thought when I proposed the solutions. That is because there should not be async code in a class constructor. There are ways to solve for this problem, though. |
ConnectWise API's base path can and will change. Furthermore, CW Manage partners may be on different codebase (even if it cloud hosted). Therefore, the best practice is to build the
basePath
after retrieving theCodebase
. This can be queried viahttps://{host}/login/companyinfo/{company-name}
.Acme Corporation (Fake) Example
The text was updated successfully, but these errors were encountered: