Project

General

Profile

Actions

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 almost 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Story points:
0.5

Description

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


Subtasks 2 (0 open2 closed)

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

Related issues 1 (0 open1 closed)

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

Updated by Radhika Chippada almost 10 years ago

  • Description updated (diff)
Actions #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 #3

Updated by Tom Clegg almost 10 years ago

  • Target version changed from Bug Triage to 2015-03-11 sprint
Actions #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 #5

Updated by Tom Clegg almost 10 years ago

  • Category set to API
Actions #6

Updated by Ward Vandewege almost 10 years ago

  • Story points changed from 0.5 to 1.0
Actions #7

Updated by Tom Clegg almost 10 years ago

  • Assigned To set to Tom Clegg
Actions #8

Updated by Ward Vandewege almost 10 years ago

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

Updated by Tom Clegg almost 10 years ago

  • Story points deleted (1.0)
Actions #10

Updated by Ward Vandewege almost 10 years ago

  • Story points set to 0.5
Actions #11

Updated by Brett Smith almost 10 years ago

9a38a7c is good to merge. Thanks.

Actions #12

Updated by Anonymous almost 10 years ago

  • Status changed from New to Resolved

Applied in changeset arvados|commit:ae8ed6d9d0e5231379564beedc08c812352b7043.

Actions

Also available in: Atom PDF