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

Ensure that the CSV Import status is updated on failed import #2046

Merged
merged 3 commits into from
Apr 6, 2021
Merged

Ensure that the CSV Import status is updated on failed import #2046

merged 3 commits into from
Apr 6, 2021

Conversation

DavisRayM
Copy link
Contributor

Changes / Features implemented

  • Return failed import async status on exception
  • Reset XForm submission count on failed import
  • Test that on failed import a response is returned

Steps taken to verify this change does what is intended

  • Added tests

Side effects of implementing this change

  • N/A

Closes #2036

@DavisRayM DavisRayM merged commit bbf3d70 into master Apr 6, 2021
@DavisRayM DavisRayM deleted the 2036-csv-import-update-status branch April 6, 2021 12:48
@DavisRayM DavisRayM mentioned this pull request Apr 21, 2021
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@denniswambua denniswambua denniswambua approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CSV Import status not updated on failed import
2 participants
@DavisRayM @denniswambua