Merge ~twom/launchpad:add-affirmation-code-of-conduct into launchpad:db-devel

[Tom Wardill]

Add affirmed column to signedcodeofconduct

[Colin Watson (cjwatson)]

review: Needs Fixing

[Colin Watson (cjwatson)]

LGTM, but let's get a quick DB review from William as usual.

review: Approve (db)

[Colin Watson (cjwatson)]

Hm, I wonder whether we should also have a constraint something like `NOT affirmed OR signing_key_fingerprint IS NULL`? We probably don't want both those things to be set.

[Colin Watson (cjwatson)]

review: Approve (db)

[William Grant (wgrant)]

review: Approve (db)

[Otto Co-Pilot (otto-copilot)]

Merging failed
https://jenkins.ols.canonical.com/online-services/job/launchpad/1837/

diff --git a/database/schema/patch-2210-34-0.sql b/database/schema/patch-2210-34-0.sql
new file mode 100644
index 0000000..fe97a37
--- /dev/null
+++ b/database/schema/patch-2210-34-0.sql
@@ -0,0 +1,15 @@
+-- Copyright 2021 Canonical Ltd.  This software is licensed under the
+-- GNU Affero General Public License version 3 (see the file LICENSE).
+
+SET client_min_messages=ERROR;
+
+ALTER TABLE signedcodeofconduct
+    ADD COLUMN affirmed boolean,
+    ADD COLUMN version text;
+
+ALTER TABLE signedcodeofconduct ADD CONSTRAINT only_one_method CHECK (NOT affirmed OR signing_key_fingerprint IS NULL);
+
+COMMENT ON COLUMN signedcodeofconduct.affirmed IS 'Code of conduct was affirmed via website interaction.';
+COMMENT ON COLUMN signedcodeofconduct.version IS 'Version of the Code of Conduct that was signed.';
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2210, 34, 0);