Skip to content

Minor documentation updates#53

Merged
brysontyrrell merged 2 commits intomacadmins:mainfrom
liquidz00:docs
Aug 2, 2024
Merged

Minor documentation updates#53
brysontyrrell merged 2 commits intomacadmins:mainfrom
liquidz00:docs

Conversation

@liquidz00
Copy link
Collaborator

Changes the following:

  • Fixes a typo in contributors/index.rst (.vent) → (.venv)
  • Changes code block directive to use console instead of shell. This was for syntax highlighting purposes, and to remain consistent throughout the documentation as this was the only instance of a shell code block.
  • Adds a note to user guide about omitting URL scheme when instantiating a JamfClient object. SDK automatically handles this, if users add this information, the following error would be thrown:
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='https', port=443): Max retries exceeded with url: /<sanitized>:443/api/v1/auth/token (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x104572ec0>: Failed to resolve 'https' ([Errno 8] nodename nor servname provided, or not known)"))

@brysontyrrell brysontyrrell merged commit 8705919 into macadmins:main Aug 2, 2024
@liquidz00 liquidz00 deleted the docs branch August 8, 2024 22:47
liquidz00 pushed a commit to liquidz00/jamf-pro-sdk-python that referenced this pull request Nov 23, 2025
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.

2 participants