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

Update date when cancelling a transaction #46

Open
lebobar opened this issue Jun 15, 2017 · 6 comments
Open

Update date when cancelling a transaction #46

lebobar opened this issue Jun 15, 2017 · 6 comments

Comments

@lebobar
Copy link

lebobar commented Jun 15, 2017

Just to have it at the top of the history, and leave the original date in the comment.

Name: Manou
Email: [email protected]

@matthieu-gendreau
Copy link

Why is there three timestamp columns in transactions ?
last_update created_at updated_at

@matthieu-gendreau
Copy link

I suggest that we use updated_at to sort and display on the account page
Then eventually in the comment use created_at or do a popover to display this information
I would advise against hardcoding information in the comment (if possible)

@tdeo
Copy link
Member

tdeo commented Jun 15, 2017

@matthieu-gendreau
Copy link

I was looking at the wrong database...
What about having a created_at and a updated_at?

@tdeo
Copy link
Member

tdeo commented Jun 15, 2017

We still have an issue of retro-compatibility.

@matthieu-gendreau
Copy link

matthieu-gendreau commented Jun 15, 2017

What about we decide that the current date is the updated_at and we create a new column created_at ?

On the old TDB we cannot cancel transactions so no problem
And I guess if we log something the created_at column should be filled by mysql?

On the new we can put an alias date -> updated_at
When a transaction is cancelled you update updated_at and do not change created_at

Would it work ?

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

No branches or pull requests

3 participants