* fix: removing hard-coded edX from DemographicsSection messages
* fix: removing hard-coded “edX” strings from IDV messages
* fix: updating ThirdPartyAuth message description to remove hard-coded “edX” references.
* fix: replacing hard-coded “edX” strings with SITE_NAME in AccountSettingsPage
* fix: conditionalizing edx-specific strings in ConfirmationModal
If the SITE_NAME is ‘edX’, then edx.org-specific strings will be used. Otherwise, more general, Open edX-appropriate strings will be used.
* fix: conditionalizing edX-specific strings in delete account components.
If the SITE_NAME is ‘edX’, then edx.org-specific strings will be used. Otherwise, more general, Open edX-appropriate strings will be used.
* fix: replacing hard-coded ‘edX’ strings with SITE_NAME in ReviewRequirementsPanel
I missed a few because the messages were re-used.
* fix: review feedback, improving messages
- Removing unnecessary {siteName} references
- Improving some message descriptions
* Mild style whitespace linting
* Updating to latest paragon and brand.
Needed to update Button/StatefulButton props to use variant, primarily.
* Adding new environment variables.
* Fixing prop-types warning.
* Updating snapshots. Modal and Button changed primarily.
I’ve reviewed the various snapshots and determined they’re all correct. The button prop changes are in line with what I’ve seen elsewhere with the new react-bootstrap-based Button component replacing our own button. The modal changes make sense, as I think we added some focus lock handling.
* Locking dependency versions in package.json
* Removing dataUtils functions, extraneous deps, and updating frontend-build
Committing all these at the same time since they affect package-lock.json together and splitting them out is nearly impossible now.
* Turning the linter on.
Hold on to your butts!
After this commit, there will be ~1600 linting errors to fix in subsequent commits.
* Main app auto-linting.
Not including coaching, demographics, or ID verification.
This is all the whitespace/syntax linting that my auto-formatter fixed. I did a few small whitespace cleanups after it, but this commit is 95% automatic.
* Removing HeaderFooterLayout
The HeaderFooterLayout was only used in one place. Collapsing it down again; also means we don’t have to have prop types for it or split it out into a separate file or anything.
* Main app propTypes cleanup
We were missing some propTypes in AccountSettingsPage and EditableField.
JumpNav had a default that was unused, since displayDemographicsLink is Required.
* Main app manual linting
AccountSettingsPage had some function-ordering issues, and some weak equality checking.
EditableField had an if without curly braces, followed by a variable named “value” which obscured a variable with the same name in the parent scope. I renamed it to “finalValue” to avoid the name reuse.
* Coaching auto-fixed lint errors
These are general whitespace and syntax linting errors that my IDE automatically fixed. I went in after and tweaked the whitespace a bit cause it didn’t finish the job, but this is 95% automatic.
* Coaching unused prop in CoachingConsent.
* Demographics auto-fixed lint errors.
Again, 95% auto-fixed linting errors, done by my IDE. I tweaked a few here and there for spacing and such where it didn’t do a perfecto job.
* Demographics Checkboxes manual linting fixes.
A few things here:
- We were double-exporting Checkboxes, once as the default, once as a named export. I removed the named export.
- Now uses === when checking string equality.
- id prop was always set, so I made it required
- PropTypes.array is not specific enough for the linter, so I found out what “values” and “options” were being set to and made some arrayOf PropTypes declarations that were more specific.
- onChange is also always set, so now it’s required.
* Demographics manual linting fixes for DemographicsSection
Bunch of things here:
- Reordered hasRetrievedDemographicsOptions and addDefaultOption to later in the file where the linter told me to put them.
- ethnicityFieldDisplay did not consistently return - only in the conditional did it return. Now it uses the conditional to set a value for enthicities and then consistently returns at the end, processing the ethnicities array.
- handleSubmit didn’t use its “values” prop
- handleSubmit was iterating over an object using a for…in loop, which iterates over ALL properties of an object, not just the keys you might expect. Probably not a problem, but not a good practice either. It now uses Object.entries to get the iterable properties of the object.
- renderDemographicsServiceIssueWarning should use a || instead of a | for an OR. It can also use ! instead of == false
- Using === for string equality in showSelfDescribe and showWorkStatusDescribe
- Using the Paragon Hyperlink component instead of an a tag. It decorates links with additional metadata noting that the link will be external.
- Adding rel=“noopener noreferrer”, which is a security fix.
- Adding missing propTypes for formValues, drafts, forwardRef, and saveMultipleSettings.
* ID Verification auto-fixed linting errors
This commit includes whitespace and syntax linting errors that were auto-fixed by my IDE, with a little manual whitespace fixing by me where it didn’t get it quite right. 95% automated.
* ID Verification circular dependency between IdVerificationContext and AccessBlocked
This commit resolves a circular dependencybetween IdVerificationContext and the AccessBlocked component.
AccessBlocked imported ERROR_REASONS from IdVerificationContext and IdVerificationContext imported AccessBlocked itself.
We resolve this by moving IdVerificationContextProvider out into its own file. Then it can safely import from AccessBlocked, and both can safely import the context and constants from IdVerificationContext.
This also sets IdVerificationContext as the default export from its file, which is responsible for the majority of the file changes in this commit, where the import shape changed.
* ID verification removing an unused import in SubmittedPanel
* Ignoring @tensorflow-models/blazeface as an unresolved import
We’re depending on an alias for @tensorflow-models/blazeface which confuses eslint. I’ve just told it to ignore the problem, since the code is valid.
* ID Verification misc manual linting fixes
There’s a number of things in this commit:
- In Camera, we’re using the function version of setState so we can ensure we’re getting the right version of shouldDetect - you’re not supposed to set state from state directly, as there’s no guarantee that it’s still correct, and you might be setting it from stale data.
- In Camera, the takePhoto function inconsistently returned a value. Returning after calling this.reset() makes it consistently return undefined.
- In Camera, the <button> was missing a type.
- In Camera, it’s also violating two accessibility rules - the video media has no caption, and there’s apparently not supposed to be an accessKey on buttons. I don’t know how to fix either of those for this code so I’m punting - I’m leaving it to the owning teams.
- IdVerificationPage should be calling Object.prototype.hasOwnProperty instead of assuming a variable has Object prototype functions.
- ImagePreview didn’t set a default prop value for the id prop - I’ve set it to undefined.
- RequestCameraAccessPanel needed a button type on the “Enable” button.
- RequestCameraAccessPanel should use === for string equality.
* ID Verification manual linting fix to rearrange methods in Camera
The linter complained about the order of methods in the Camera component. This commit rearranges them to suit it.
* Ignoring module.config.js file.
* Fixing package-lock.json after rebase.
Fixed padding on 'next' button and collapsed camera help text
Added functionality to submit button
Styling edit to differentiate cards
Removed todo comments
Replaced hardcoded text with i18n components
Added title to confirmation button for a/b testing purposes
Redirect the user to the beginning of the process on reload to avoid losing camera access, also removed unused code
Update src/id-verification/IdVerificationContext.jsx, make children prop required
Co-authored-by: Kyle McCormick <kmccormick@edx.org>
Remove todo comment in ImageFileUpload.jsx
Added comment to service.js and removed unused code from index.js
Fix 404 page from displaying when it shouldn't
Was displaying below the settings page itself
Add panels for each step of the verification flow
Add IdVerificationContext
Add BasePanel
Revert "Add BasePanel"
This reverts commit 2ddca14b4b8b35cd23f0525f49070753c09d361d.
Add BasePanel
Focus heading elements on mount
Add current account name to GetNameIdPanel
Fix nested button focus problem
Add a friendly camera permission request
Remove double ctas
lint
Add initial service.js file; not tested
Add image file uploads
fix temp heading of photo context panel
Add photo id name input
Add redirect logic for users who find themselves too far in the flow
add wip SubmittedPanel
Wire up Submit button
Update routing-utilities.js
Preview image on summarypenl; still buggy
Add some content and styles
update first panel content
Update headings
Use ImagePreview component on SummaryPanel
Fix service.js; it works now
More content and fix for name input
ImagePreview: Change 'name' to 'alt'
Add content to camera request
Fix ImagePreview alt text
Iterating on SummaryPage
Add privacy info modal
Update name entry
Make the footer sticky to the bottom
Update upload photo screens
Add photo to name panel
camera component
overlay camera button
implement camera component
remove camera tracking
Add retake photo buttons
Add service for verification status endpoint
delete old css
lint fix
Better name edit state
portrait photo page
Add edit name from summary
Add content to submitted screen
lint
Rename status service function
Add clarity to summary view text
First pass at gating based on existing idv
Clean up IDv-status-gating screens
finish up webcam/photo pages
remove unused tracking library
