-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Improve social authentication #5349
Conversation
/check |
❌ Some checks failed |
231ee10
to
e39850b
Compare
/check |
✔️ All checks completed successfully |
}, | ||
}; | ||
cvat.server.request( | ||
`${cvat.config.backendAPI}/auth/${provider}/login/token?state=${state}`, req, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Marishka17 , should it be a part of cvat-core?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean dedicated API method like loginSocial
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Marishka17 , it looks good from my perspective at least. Need to slightly change the code. Also I expect to see all these auth methods in our swagger. What is a reason to hide them?
cvat-ui/src/components/login-with-social-app/login-with-social-app.tsx
Outdated
Show resolved
Hide resolved
}, | ||
}; | ||
cvat.server.request( | ||
`${cvat.config.backendAPI}/auth/${provider}/login/token?state=${state}`, req, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean dedicated API method like loginSocial
?
cvat-ui/src/components/login-with-social-app/login-with-social-app.tsx
Outdated
Show resolved
Hide resolved
cvat-ui/src/components/login-with-social-app/login-with-social-app.tsx
Outdated
Show resolved
Hide resolved
Depends on cvat-ai#5349 Related cvat-ai#5432 Added tests for social account authentication functionality: cypress test with dummy auth server
Motivation and context
How has this been tested?
In another PR
Checklist
develop
branch- [ ] I have added a description of my changes into CHANGELOG file- [ ] I have updated the documentation accordingly- [ ] I have linked related issues (read github docs)cvat-core, cvat-data and cvat-ui)
License
Feel free to contact the maintainers if that's a concern.