Add more videos for the Basic Algorithms challenges.
Update the challenge title to a) not render the mobile caret icon and b) not render the editable text outside of Codepress to try to avoid the error (see below notes).
Notes:
I found that the app fails to load in a FB app generated web preview, and produces an error report like this:
{
componentStack:
in Blueprint3.EditableText
in t
in WorkspaceComponents__StyledEditableText
in div
in h1
in t
in WorkspaceComponents__ChallengeTitleHeading
in div
in Unknown
in d
in div
in div
in t
in WorkspaceComponents__WorkspaceMobileView
in div
in t
in WorkspaceComponents__WorkspaceContainer
in div
in t
in WorkspaceComponents__PageSection
in div
in t
in WorkspaceComponents__Container
in t
in t
in d
in t
in t
in Unknown
in d
in div
in f
in t
in d
in t
,
error: {
message: Minified React error #185; visit https://reactjs.org/docs/error-decoder.html?invariant=185 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.,
name: Error,
stack:
Error: Minified React error #185; visit https://reactjs.org/docs/error-decoder.html?invariant=185 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
We have a lot of these errors in Sentry, so they may be coming from people opening the app on mobile devices. I'm not sure exactly how/why, but that stacktrace appears to indicate it has something to do with the EditableText component, which we don't need to render outside of Codepress. I removed that in the PR here, and replaced it with the regular Text component instead.
This PR:
Notes:
I found that the app fails to load in a FB app generated web preview, and produces an error report like this:
We have a lot of these errors in Sentry, so they may be coming from people opening the app on mobile devices. I'm not sure exactly how/why, but that stacktrace appears to indicate it has something to do with the
EditableText
component, which we don't need to render outside of Codepress. I removed that in the PR here, and replaced it with the regularText
component instead.