Skip to content
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.

Sign up for GitHub

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

Jump to bottom

SyntaxError: "2" contains no units #7087

Closed
nithusha21 opened this issue Jul 3, 2019 · 0 comments · Fixed by #7157
Closed

SyntaxError: "2" contains no units #7087

nithusha21 opened this issue Jul 3, 2019 · 0 comments · Fixed by #7157
Labels
server errors

Comments

@nithusha21
Copy link
Contributor

This error occurred recently in production

SyntaxError: "2" contains no units
at Function.$.parse (third_party.min.a8d0c323da3c9efa572d2a025c4ef104.js:1)
at string (https://www.oppia.org/build/third_party/generated/js/third_party.min.a8d0c323da3c9efa572d2a025c4ef104.js:1)
at Object.unit (https://www.oppia.org/build/third_party/generated/js/third_party.min.a8d0c323da3c9efa572d2a025c4ef104.js:1)
at p (https://www.oppia.org/build/extensions/interactions/NumberWithUnits/directives/NumberWithUnitsValidationService.36fef879b50e047048960385c1e4baab.js:1)
at Object.getAllWarnings (https://www.oppia.org/build/extensions/interactions/NumberWithUnits/directives/NumberWithUnitsValidationService.36fef879b50e047048960385c1e4baab.js:1)
at (https://www.oppia.org/build/templates/head/pages/exploration_editor/ExplorationWarningsService.45e5d7fbc44909d55ff4c6d6ed6f3208.js:1)
at Array.forEach (<anonymous>)
at k (https://www.oppia.org/build/templates/head/pages/exploration_editor/ExplorationWarningsService.45e5d7fbc44909d55ff4c6d6ed6f3208.js:1)
at Object.updateWarnings (https://www.oppia.org/build/templates/head/pages/exploration_editor/ExplorationWarningsService.45e5d7fbc44909d55ff4c6d6ed6f3208.js:1)
at (https://www.oppia.org/build/templates/head/pages/exploration_editor/ExplorationEditor.e68a40ea020b62c50cf21b6f6b9690c3.js:1)

Additional context Add details from the logs, like the page the error occurred on. Make sure to anonymize user data like user IDs, exploration IDs, state names, user names, etc. Replace the personal data with appropriate placeholders.

General instructions
There are no specific repro steps available for this bug report. The general procedure to fix server errors should be the following:

  • Analyze the code in the file where the error occurred and come up with a hypothesis for the reason.
  • Get the logic of the proposed fix validated by an Oppia team member (have this discussion on the issue thread).
  • Make a PR that fixes the issue, then close the issue on merging the PR. (If the error reoccurs in production, the issue will be reopened for further investigation.)
kevintab95 pushed a commit to kevintab95/oppia that referenced this issue Jul 17, 2019
adds log for oppia#7087
6031a26
@kevintab95 kevintab95 mentioned this issue Jul 17, 2019
Merged
8 tasks
seanlip pushed a commit that referenced this issue Jul 18, 2019
@seanlip
Fix #7087: Add log for "contains no units" SyntaxError (#7157)
1ec3f47 
* Fixes #2197 and #2195

* fix e2e error (library)

* Use URL as constants

* Constants in all caps

* Fix linting issue

* adds log for #7087

* address review comment

* address review comment

* log earlierInput
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
server errors
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #7087: Add log for "contains no units" SyntaxError kevintab95/oppia
1 participant
@nithusha21