-
Notifications
You must be signed in to change notification settings - Fork 350
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
fix(LEMS-2738): Convert Expression Editor Answer keys to strings #2042
Conversation
Size Change: +49 B (0%) Total Size: 1.27 MB
ℹ️ View Unchanged
|
npm Snapshot: PublishedGood news!! We've packaged up the latest commit from this PR (1577d98) and published it to npm. You Example: yarn add @khanacademy/perseus@PR2042 If you are working in Khan Academy's webapp, you can run: ./dev/tools/bump_perseus_version.sh -t PR2042 |
73fa4c7
to
1577d98
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.
I have one comment on a line that looked odd that I have questions about. But once that's resolved it looks good to go.
@@ -565,6 +570,7 @@ class AnswerOption extends React.Component< | |||
|
|||
handleImSure = () => { | |||
this.props.onDelete(); | |||
this.handleCancelDelete(); |
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.
Ooo, what's this do and why is it here?
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.
Its an existing function that closes the "I'm sure" button, I added it so it wouldn't stay open once someone deletes the current item
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.
Here is the before:
https://github.com/user-attachments/assets/8838f716-dfd4-4aa7-995d-b15f4c5aa22a
Here is the after:
Screen.Recording.2024-12-19.at.17.00.15.mov
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @khanacademy/perseus-editor@17.0.11 ### Patch Changes - [#2042](#2042) [`705d74b69`](705d74b) Thanks [@anakaren-rojas](https://github.com/anakaren-rojas)! - fix for editor Author: khan-actions-bot Reviewers: anakaren-rojas Required Reviewers: Approved By: anakaren-rojas Checks: ⏭️ Publish npm snapshot, ✅ Lint, Typecheck, Format, and Test (ubuntu-latest, 20.x), ✅ Cypress (ubuntu-latest, 20.x), ✅ Check for .changeset entries for all changed files (ubuntu-latest, 20.x), ✅ Publish Storybook to Chromatic (ubuntu-latest, 20.x), ✅ Check builds for changes in size (ubuntu-latest, 20.x), ✅ Prime node_modules cache for primary configuration (ubuntu-latest, 20.x) Pull Request URL: #2051
Summary:
onDelete
handler to use index instead of keyIssue: LEMS-2738
Test plan:
Screen.Recording.2024-12-19.at.16.18.48.mov