-
Notifications
You must be signed in to change notification settings - Fork 147
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
feat: Add embeddable registration form #1026
feat: Add embeddable registration form #1026
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## embeddable-registration-form #1026 +/- ##
================================================================
+ Coverage 84.51% 85.21% +0.70%
================================================================
Files 113 114 +1
Lines 2195 2326 +131
Branches 631 664 +33
================================================================
+ Hits 1855 1982 +127
- Misses 324 328 +4
Partials 16 16
☔ View full report in Codecov by Sentry. |
d5d3fcd
to
3dd17f9
Compare
3dd17f9
to
cac54f7
Compare
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.
Although the coverage is increasing but if you see your file changes, there are alot of lines not covered by tests. please add (copy paste) their tests too.
cac54f7
to
8897992
Compare
68532fc
to
b02192e
Compare
src/register/RegistrationPage.jsx
Outdated
|
||
if (name === 'name') { | ||
validateInput( | ||
name, | ||
value, | ||
{ name: formFields.name, username: formFields.username, form_field_key: name }, | ||
!validationApiRateLimited, | ||
false, | ||
); |
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.
This complete functionality was added for embedded registration. We don't need it for registration page: https://github.com/openedx/frontend-app-authn/pull/950/files
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.
addressed
Description: Adding embedded registration form VAN-1574
b02192e
to
247e6ab
Compare
90db7ba
into
embeddable-registration-form
Description: Adding embedded registration form VAN-1574
Description:
Adding embedded registration form
VAN-1574
How Has This Been Tested?
It has been tested locally.