Skip to content

docs: standardize Auth0FastAPI import path in README#101

Merged
frederikprijck merged 1 commit into
mainfrom
fix/readme-import-paths
Jun 3, 2026
Merged

docs: standardize Auth0FastAPI import path in README#101
frederikprijck merged 1 commit into
mainfrom
fix/readme-import-paths

Conversation

@frederikprijck
Copy link
Copy Markdown
Member

@frederikprijck frederikprijck commented Jun 3, 2026

Summary

  • Fix broken import on README line 188 (from auth0_fastapi_api import Auth0FastAPI) — there is no auth0_fastapi_api module, so this raised ModuleNotFoundError.
  • Standardize all four Auth0FastAPI imports to the top-level re-export from fastapi_plugin import Auth0FastAPI (previously a mix of fastapi_plugin.fast_api_client and the broken path).

Auth0FastAPI is defined in fastapi_plugin/fast_api_client.py and re-exported from fastapi_plugin/__init__.py, so the top-level import is the cleanest correct form.

Test plan

  • Verify each README code block imports from fastapi_plugin import Auth0FastAPI

Fix the broken `auth0_fastapi_api` import and use the top-level
`from fastapi_plugin import Auth0FastAPI` across all examples.
@frederikprijck frederikprijck requested a review from a team as a code owner June 3, 2026 12:12
Copy link
Copy Markdown
Contributor

@kishore7snehil kishore7snehil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@frederikprijck frederikprijck merged commit d448b4d into main Jun 3, 2026
5 of 9 checks passed
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