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.

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 "Save" button on Exemplars not saving #54624

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

kobryan0619
Copy link
Contributor

@kobryan0619 kobryan0619 commented Nov 1, 2023

I think I fixed Ken's reported issue of the LevelBuilder "Save" button not working when editing the exemplars in JavaLab.

It is not clear to me how it got broken in the first place though which makes me a bit nervous. I checked the last time any of the impacted files were edited and none of them were less than 4 months ago...

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@kobryan0619 kobryan0619 marked this pull request as ready for review November 1, 2023 16:06
Copy link
Member

@davidsbailey davidsbailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great find @kobryan0619 ! It took me a minute of staring at your diff to realize what had even changed here

@kobryan0619 kobryan0619 merged commit b6df960 into staging Nov 1, 2023
2 checks passed
@kobryan0619 kobryan0619 deleted the kaitie/edit-exemplar-button-save branch November 1, 2023 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants