Skip to content

Add support for the Admin Audit Events API #120

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

Merged
merged 13 commits into from
Jul 7, 2020
Merged

Add support for the Admin Audit Events API #120

merged 13 commits into from
Jul 7, 2020

Conversation

cmlccie
Copy link
Collaborator

@cmlccie cmlccie commented Jul 7, 2020

  • Add support for the Admin Audit Events API
  • Update the Travis CI jobs
  • Use pytest marking to mark the slow rate-limit test and to mark the current tests that are failing due to issues with the test account
  • Squash a timezone info bug that was affecting Python v2.7 and 3.5

cmlccie added 13 commits June 19, 2020 12:42
Webex isn't allowing accounts in the trial org to be set as moderators via the REST APIs.

Mark the tests as expected to fail while we work to resolve the test account issues.
Add the API wrapper, data model, tests, and update the docs.
Use of f-strings means this example will only work with Python versions 3.6+
Lint once using the latest version of Python. Run the test suite against all supported Python distributions.
Attempting to apply a timezone to a naive datetime (no tzinfo) raises a Value error in Python v2.7 and v3.5.

ValueError: astimezone() cannot be applied to a naive datetime

Check to see if the datetime has time zone information (tzinfo). If it has timezone info, use the `astimezone()` method to adjust it to ZuluTime.  If it does not have timezone info, assume the time is UTC time and add the ZuluTimeZone (raise a warning when this happens).
@cmlccie cmlccie closed this Jul 7, 2020
@cmlccie cmlccie reopened this Jul 7, 2020
@cmlccie cmlccie merged commit 704904a into master Jul 7, 2020
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.

1 participant