Code review comment for lp:~al-maisan/launchpad/duration-485524

Revision history for this message
Muharem Hrnjadovic (al-maisan) wrote :

Stuart Bishop wrote:
> Review: Approve db
> Seems fine. patch-2207-14-0.sql
>
> You are citing a rather out of date spec - it is still listing three possible implementations and there is no mention of the column you are adding at all or which of the three possibilities has been chosen.

Hello Stuart,

thank you very much for reviewing this. Sorry for not being more
precise, this patch "builds on" patch-2207-11-0.sql which implements the
bulk of the schema changes for the outdated spec.

Best regards

--
Muharem Hrnjadovic <email address hidden>
Public key id : B2BBFCFC
Key fingerprint : A5A3 CC67 2B87 D641 103F 5602 219F 6B60 B2BB FCFC

« Back to merge proposal