Bug #5207

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

Added by Radhika Chippada almost 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Start date:
03/12/2015
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Description

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


Subtasks

Task #5455: MigrationResolvedTom Clegg

Task #5457: Review 5207-expires-at-timeResolvedTom Clegg


Related issues

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

Associated revisions

Revision ae8ed6d9
Added by Tom Clegg over 5 years ago

Merge branch '5207-expires-at-time' closes #5207

History

#1 Updated by Radhika Chippada almost 6 years ago

  • Description updated (diff)

#2 Updated by Tom Clegg almost 6 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.

#3 Updated by Tom Clegg almost 6 years ago

  • Target version changed from Bug Triage to 2015-03-11 sprint

#4 Updated by Tom Clegg almost 6 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.

#5 Updated by Tom Clegg almost 6 years ago

  • Category set to API

#6 Updated by Ward Vandewege almost 6 years ago

  • Story points changed from 0.5 to 1.0

#7 Updated by Tom Clegg almost 6 years ago

  • Assigned To set to Tom Clegg

#8 Updated by Ward Vandewege almost 6 years ago

  • Target version changed from 2015-03-11 sprint to 2015-04-01 sprint

#9 Updated by Tom Clegg over 5 years ago

  • Story points deleted (1.0)

#10 Updated by Ward Vandewege over 5 years ago

  • Story points set to 0.5

#11 Updated by Brett Smith over 5 years ago

9a38a7c is good to merge. Thanks.

#12 Updated by Anonymous over 5 years ago

  • Status changed from New to Resolved

Applied in changeset arvados|commit:ae8ed6d9d0e5231379564beedc08c812352b7043.

Also available in: Atom PDF