Project

General

Profile

Actions
Copy link

Bug #5207

closed

[API] Collection expires_at should be a full timestamp, not just a date.

Added by Radhika Chippada almost 10 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Tom Clegg
Category:
API
Target version:
2015-04-01 sprint
Story points:
0.5

Description

Is there a reason why it is set as Date instead of timestamp?

Subtasks 2 (0 open — 2 closed)

Task #5455: MigrationResolvedTom Clegg03/12/2015Actions
Task #5457: Review 5207-expires-at-timeResolvedTom Clegg03/12/2015Actions

Related issues

Related to Arvados - Idea #3150: [API] Add "trash" behavior for all object types, instead of literal rdbms "delete"Rejected07/04/2014Actions
Actions
Copy link
 #1

Updated by Radhika Chippada almost 10 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Tom Clegg almost 10 years ago

expires_at should absolutely be a timestamp just like all the others.

Suggest a unit test that verifies every *_at column is a timestamp column, and vice versa.

Actions
Copy link
 #3

Updated by Tom Clegg almost 10 years ago

  • Target version changed from Bug Triage to 2015-03-11 sprint
Actions
Copy link
 #4

Updated by Tom Clegg almost 10 years ago

  • Story points set to 0.5
  • Subject changed from [API] Collection expires_at is a date type. It would be much more helpful if it is a timestamp and has the time at which a collection has expired. to [API] Collection expires_at should be a full timestamp, not just a date.
Actions
Copy link
 #5

Updated by Tom Clegg almost 10 years ago

  • Category set to API
Actions
Copy link
 #6

Updated by Ward Vandewege almost 10 years ago

  • Story points changed from 0.5 to 1.0
Actions
Copy link
 #7

Updated by Tom Clegg almost 10 years ago

  • Assigned To set to Tom Clegg
Actions
Copy link
 #8

Updated by Ward Vandewege almost 10 years ago

  • Target version changed from 2015-03-11 sprint to 2015-04-01 sprint
Actions
Copy link
 #9

Updated by Tom Clegg over 9 years ago

  • Story points deleted (1.0)
Actions
Copy link
 #10

Updated by Ward Vandewege over 9 years ago

  • Story points set to 0.5
Actions
Copy link
 #11

Updated by Brett Smith over 9 years ago

9a38a7c is good to merge. Thanks.

Actions
Copy link
 #12

Updated by Anonymous over 9 years ago

  • Status changed from New to Resolved

Applied in changeset arvados|commit:ae8ed6d9d0e5231379564beedc08c812352b7043.

Actions
Copy link

Also available in: Atom PDF