-
Notifications
You must be signed in to change notification settings - Fork 13.1k
📝 Add docstrings to fix/third-party-login
#37741
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
📝 Add docstrings to fix/third-party-login
#37741
Conversation
Docstrings generation was requested by @d-gubert. * #37707 (comment) The following files were modified: * `apps/meteor/app/api/server/middlewares/authentication.ts` * `apps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.ts`
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
|
|
Important Review skippedCodeRabbit bot authored PR detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
There will be changes to the signatures, closing this as a new one will be required |
Docstrings generation was requested by @d-gubert.
The following files were modified:
apps/meteor/app/api/server/middlewares/authentication.tsapps/meteor/app/oauth2-server-config/server/oauth/oauth2-server.tsThese file types are not supported
.changeset/rich-dogs-wonder.mdℹ️ Note