Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

clear completedAt when retrying exports #3086

Merged
merged 1 commit into from Mar 4, 2022
Merged

clear completedAt when retrying exports #3086

merged 1 commit into from Mar 4, 2022

Conversation

pedroslopez
Copy link
Member

@pedroslopez pedroslopez commented Mar 4, 2022 •
edited

Change description

Also clears out the completedAt field

Checklists

Development

  • Application changes have been tested appropriately

Impact

  • Code follows company security practices and guidelines
  • Security impact of change has been considered
  • Performance impact of change has been considered
  • Possible migration needs considered (model migrations, config migrations, etc.)

Please explain any security, performance, migration, or other impacts if relevant:

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached where applicable.
  • Relevant tags have been added to the PR (bug, enhancement, internal, etc.)

@pedroslopez pedroslopez added the bug Something isn't working label Mar 4, 2022
@pedroslopez pedroslopez merged commit ce147c5 into main Mar 4, 2022
@pedroslopez pedroslopez deleted the 181314133-retry-no-completedAt branch March 4, 2022 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@krishnaglick krishnaglick krishnaglick approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pedroslopez @krishnaglick