Skip to content

Enhancement: Add deleted status to projects table - #5730

Open
pmedinar01 wants to merge 2 commits into
masterfrom
deleted-status-column
Open

Enhancement: Add deleted status to projects table#5730
pmedinar01 wants to merge 2 commits into
masterfrom
deleted-status-column

Conversation

@pmedinar01

Copy link
Copy Markdown
Contributor

No description provided.

@glpatcern glpatcern left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As this is a GORM object, an entry is deleted if deleted_at is not NULL. Why would we need an additional status? and why would we override the GORM one?

@pmedinar01

Copy link
Copy Markdown
Contributor Author

e?

As we might receive status changes to deleted from the resource portal ... we thought it was a way to keep track of precisely that.

@diocas

@glpatcern

Copy link
Copy Markdown
Member

As we might receive status changes to deleted from the resource portal ... we thought it was a way to keep track of precisely that.

But then what is the workflow we're trying to address? We always agreed that for us, a "delete" action = we move to archived, we never delete from a user action. We have a delete in case of mistakes as you know.

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.

2 participants