Skip to content

Conversation

@luiscastillocr
Copy link

Renamed from requests.py to api.py to avoid name ambiguity with the requests package
Moved exceptions to a separate file to avoid future circular references
Modified CalendlyReq to raise CalendlyException exception if the requests response returns a 400 or higher
New CalendlyOauth2 class
Added TestCalendlyReq and TestCalendlyOauth2 to run_tests.py

…ts package

Moved exceptions to a separate file to avoid future circular references
Modified CalendlyReq to raise CalendlyException exception if the requests response returns a 400 or higher
New CalendlyOauth2 class
Added TestCalendlyReq and TestCalendlyOauth2 to run_tests.py
…eturn the proper exception type(CalendlyOauth2Exception)
@laxmena
Copy link
Owner

laxmena commented Mar 15, 2022

Update: Will review the changes and update the pull request within 2 days.

@luiscastillocr
Copy link
Author

BUMP!

@SinaKarimi7
Copy link

Update: Will review the changes and update the pull request within 2 days.

Is there any update to this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants