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

Bug: multiple changes in quick succession on new entry cause exception #1758

Conversation

myieye
Copy link
Collaborator

@myieye myieye commented Jun 19, 2023

Fixes #1757

Description

See issue

Checklist

  • I have labeled my PR with: bug, feature, engineering, security fix or testing
  • I have performed a self-review of my own code
  • I have reviewed the title & description of this PR which I will use as the squashed PR commit message
  • I have commented my code, particularly in hard-to-understand areas
    I have added tests that prove my fix is effective or that my feature works
  • I have enabled auto-merge (optional)

Testing

Testers, use the following instructions against our staging environment. Post your findings as a comment and include any meaningful screenshots, etc.

Steps to reproduce the original error:

  • Go to the Editor
  • Click on '+' (Create new entry). Note: an entry only actually gets created when some data is added to it.
  • Quickly click on "Add meaning" and then "Add example"
  • See error

@myieye myieye added the bug An existing problem with our app in production label Jun 19, 2023
@github-actions
Copy link

github-actions bot commented Jun 19, 2023

Unit Test Results

362 tests   362 ✔️  16s ⏱️
  37 suites      0 💤
    1 files        0

Results for commit 7987f4d.

♻️ This comment has been updated with latest results.

@myieye myieye force-pushed the bug/1757-bug-multiple-changes-in-quick-succession-on-new-entry-cause-exception branch from a9b125a to f5512c9 Compare June 19, 2023 13:22
@myieye myieye merged commit 3b0dd5d into develop Jun 20, 2023
@myieye myieye deleted the bug/1757-bug-multiple-changes-in-quick-succession-on-new-entry-cause-exception branch June 20, 2023 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An existing problem with our app in production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Multiple changes in quick succession on new entry cause exception
1 participant