deis / deis/workflow-manager-api

Investigate whether clustersCheckinsTable can use a *Timestamp

Open
#118 3 comments 0 reactions 0 assignees View on GitHub
bug refactor
Dominant language
Go
Stars
5
Forks
0
PR merge metrics
No merged PRs in 30d

Description

it's currently using a `string` because at the time, it looked like gorm couldn't handle `(database/sql/driver).Valuer`s. I've found, in https://github.com/deis/workflow-manager-api/pull/105, that it can, however. Investigate whether it can use a `*Timestamp` instead (which itself is a `Valuer`)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.