Skip to content
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

Auth0 provider docs patch #724

Closed
wants to merge 3 commits into from
Closed

Conversation

waxmar
Copy link
Contributor

@waxmar waxmar commented Jan 24, 2024

Note

This is my very first contribution to open source.

Description

Fixed this syntax error in docs/functions/Auth0Provider.html
Before:
image
After:
image

Removed the additional } and applied the matching css classes.

Testing

Opening it in a browser to verify the layout. It was checked locally with Chrome Version 120.0.6099.234

  • This change adds test coverage for new/changed/fixed functionality

Checklist

  • I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • All active GitHub checks for tests, formatting, and security are passing -> couldn't find out how to run them
  • The correct base branch is being used, if not the default branch

@frederikprijck
Copy link
Member

frederikprijck commented Jan 24, 2024

Thanks for your contribution!

The docs is autogenerated, so we should not modify the content of the docs folder. You want to patch this line: https://github.com/auth0/auth0-react/blob/main/src/auth0-provider.tsx#L128 instead.

@waxmar
Copy link
Contributor Author

waxmar commented Jan 25, 2024

Thanks for your help, I'll close this pull request and make a new one.

@waxmar waxmar closed this Jan 25, 2024
@waxmar waxmar deleted the auth0-provider-docs-patch branch January 25, 2024 09:11
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