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

Deprecate DocumentStatus field in favour of DeletedAt #1123

Closed
danvixent opened this issue Nov 6, 2022 · 0 comments · Fixed by #1125
Closed

Deprecate DocumentStatus field in favour of DeletedAt #1123

danvixent opened this issue Nov 6, 2022 · 0 comments · Fixed by #1125
Assignees
@danvixent

Comments

@danvixent
Copy link
Collaborator

The DocumentStatus in our models is becoming an issue & it hard to keep track of alongside DeletedAt, it should be deprecated so we only use DeletedAt to identify deleted records.
@danvixent danvixent self-assigned this Nov 6, 2022
@danvixent danvixent changed the title Deprecate DocumentStatus field in favour of DeletedAt field Deprecate DocumentStatus field in favour of DeletedAt Nov 6, 2022
@danvixent danvixent mentioned this issue Nov 6, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danvixent danvixent

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

deprecate document_status field frain-dev/convoy
1 participant
@danvixent