Revert "chore: Finalize Reviews DB schema."

This reverts commit 139490e.

Author: kukkok3

src/event-db/migrations/V11__reviews.sql

@@ -5,12 +5,12 @@
 
 CREATE TABLE subscription
 (
-  row_id    SERIAL PRIMARY KEY,
-  user_id   INTEGER NOT NULL,
-  event_id  INTEGER NOT NULL,
-  role      INTEGER NOT NULL,
-  status    INTEGER NOT NULL,
-  extra     JSONB NULL,
+  row_id SERIAL PRIMARY KEY,
+  user_id INTEGER NOT NULL,
+  event_id INTEGER NOT NULL,
+  role INTEGER NOT NULL,
+  status INTEGER NOT NULL,
+  extra  JSONB NULL,
 
   FOREIGN KEY (user_id) REFERENCES catalyst_user(row_id) ON DELETE CASCADE,
   FOREIGN KEY (event_id) REFERENCES event(row_id) ON DELETE CASCADE
@@ -27,9 +27,6 @@ Some of these subscriptions will be automatically created by the system.
 The presence/status of subscriptions will determine user capabilities in the app.
 ';
 
-COMMENT ON COLUMN subscription.user_id IS 'The user ID this subscription belongs to.';
-COMMENT ON COLUMN subscription.event_id IS 'The event ID this subscription belongs to.';
-
 COMMENT ON COLUMN subscription.role IS
 'This field describes the role of the user for this subscription.
 Possible values:
@@ -43,8 +40,7 @@ Specifically:
 
 anonymous_id: str,
 subscription_date: datetime,
-preferred_categories: [int],
-reward_address: str
+preferred_categories: [int]
 ';
 
 
@@ -55,11 +51,11 @@ reward_address: str
 -- - the allocation of moderations that needs to be done.
 
 CREATE TABLE allocation (
-  row_id      SERIAL PRIMARY KEY,
+  row_id SERIAL PRIMARY KEY,
   proposal_id INTEGER NULL,
-  review_id   INTEGER NULL,
-  user_id     INTEGER NOT NULL,
-  type        INTEGER NOT NULL,
+  review_id INTEGER NULL,
+  user_id INTEGER NOT NULL,
+  type INTEGER NOT NULL,
 
   FOREIGN KEY (proposal_id) REFERENCES proposal(row_id) ON DELETE CASCADE,
   FOREIGN KEY (review_id) REFERENCES proposal_review(row_id) ON DELETE CASCADE,
@@ -69,9 +65,9 @@ CREATE TABLE allocation (
 
 COMMENT ON TABLE allocation IS 'The relationship between users and proposals or proposals_reviews.';
 COMMENT ON COLUMN allocation.row_id IS 'Synthetic ID of this relationship.';
-COMMENT ON COLUMN allocation.proposal_id IS 'The proposal ID the relationship belongs to.';
-COMMENT ON COLUMN allocation.review_id IS 'The review ID the relationship belongs to.';
-COMMENT ON COLUMN allocation.user_id IS 'The user ID the relationship belongs to.';
+COMMENT ON COLUMN allocation.proposal_id IS 'The proposal the relationship is related to.';
+COMMENT ON COLUMN allocation.review_id IS 'The review the relationship is related to.';
+COMMENT ON COLUMN allocation.user_id IS 'The user the relationship is related to.';
 COMMENT ON COLUMN allocation.type IS 'The type of relationship stored.
 Possible values:
 0: proposal ownership relation. proposal_id and user_id are required
@@ -86,11 +82,11 @@ CREATE INDEX idx_allocation_user_type ON allocation(user_id, type);
 -- Moderation - Defines the moderation submitted by users for each proposal_review.
 
 CREATE TABLE moderation (
-  row_id          SERIAL PRIMARY KEY,
-  review_id       INTEGER NOT NULL,
-  user_id         INTEGER NOT NULL,
-  classification  INTEGER NOT NULL,
-  rationale       VARCHAR,
+  row_id SERIAL PRIMARY KEY,
+  review_id INTEGER NOT NULL,
+  user_id INTEGER NOT NULL,
+  classification INTEGER NOT NULL,
+  rationale VARCHAR,
   UNIQUE (review_id, user_id),
 
   FOREIGN KEY (review_id) REFERENCES proposal_review(row_id) ON DELETE CASCADE,
@@ -100,7 +96,7 @@ CREATE TABLE moderation (
 
 COMMENT ON TABLE moderation IS 'An individual moderation for a proposal review.';
 COMMENT ON COLUMN moderation.row_id IS 'Synthetic ID of this moderation.';
-COMMENT ON COLUMN moderation.review_id IS 'The review ID the moderation belongs to.';
-COMMENT ON COLUMN moderation.user_id IS 'The user ID the moderation belongs to.';
+COMMENT ON COLUMN moderation.review_id IS 'The review the moderation is related to.';
+COMMENT ON COLUMN moderation.user_id IS 'The user the moderation is submitted from.';
 COMMENT ON COLUMN moderation.classification IS 'The value used to describe the moderation (e.g. 0: excluded, 1: included).';
 COMMENT ON COLUMN moderation.rationale IS 'The rationale for the given classification.';

src/event-db/migrations/V4__catalyst_id.sql

@@ -5,41 +5,21 @@
 
 CREATE TABLE catalyst_id_map
 (
-  row_id                    SERIAL PRIMARY KEY,
-  catalyst_id               VARCHAR NOT NULL,
-  email                     VARCHAR NOT NULL,
-  username                  VARCHAR NOT NULL,
-  signed_email              VARCHAR NOT NULL,
-  status                    INTEGER NOT NULL,
-  confirmation_token        VARCHAR, 
-  confirmation_requested_at TIMESTAMP,
-  confirmed_at              TIMESTAMP,
+  row_id SERIAL PRIMARY KEY,
+  catalyst_id VARCHAR NOT NULL,
+  email VARCHAR NOT NULL,
+  signed_email  VARCHAR NOT NULL,
+  status  INTEGER NOT NULL,
+  confirmation_token VARCHAR NOT NULL, 
 
   UNIQUE(catalyst_id)
 );
 
 CREATE UNIQUE INDEX catalyst_id_idx ON catalyst_id_map(catalyst_id, email);
 
-COMMENT ON TABLE catalyst_id_map IS '
-The `catalyst_id_map` table is used to store the map between the CatalystID and email/username for each user.
-Because the Catalyst RBAC registration are stored on-chain, and for this reason are publicly accessible,
-sensitive information like email address will be stored in a centralized DB, keeping a reference to the CatalystID.
-The email address is stored only when its signature corresponds to the CatalystID.
-';
-COMMENT ON COLUMN catalyst_id_map.catalyst_id IS '
-It contains the unique part of the CatalystID.
-Given a full CatalystID like `id.catalyst://[email protected]/FftxFnOrj2qmTuB2oZG2v0YEWJfKvQ9Gg8AgNAhDsKE`
-The scheme and the username are omitted and only the unique part `preprod.cardano/FftxFnOrj2qmTuB2oZG2v0YEWJfKvQ9Gg8AgNAhDsKE`
-is stored in this field.
-';
-COMMENT ON COLUMN catalyst_id_map.email IS 'The email address stored in plaintext.';
-COMMENT ON COLUMN catalyst_id_map.username IS 'The username extracted from the CatalystID stored in plaintext.';
-COMMENT ON COLUMN catalyst_id_map.signed_email IS 'The signed document that includes the email address.';
+COMMENT ON COLUMN catalyst_id_map.email IS 'The email is stored encrypted.';
 COMMENT ON COLUMN catalyst_id_map.status IS '
 Describes the status of an account:
 0: Inactive
 1: Active
-2: Banned.';
-COMMENT ON COLUMN catalyst_id_map.confirmation_token IS 'The token that is generated for the email confirmation.';
-COMMENT ON COLUMN catalyst_id_map.confirmation_requested_at IS 'The timestamp to validate confirmation token validity.';
-COMMENT ON COLUMN catalyst_id_map.confirmed_at IS 'The timestamp of the email confirmation.';
+2: Banned.';

src/event-db/migrations/V4__proposal_tables.sql

@@ -101,7 +101,10 @@ CREATE TABLE proposal_review (
   proposal_id INTEGER NOT NULL,
   assessor VARCHAR NOT NULL,
   assessor_level INTEGER,
+  reward_address TEXT,
 
+  -- These fields are deprecated and WILL BE removed in a future migration.
+  -- They MUST only be used for Vit-SS compatibility.
   impact_alignment_rating_given INTEGER,
   impact_alignment_note VARCHAR,
   feasibility_rating_given INTEGER,
@@ -117,23 +120,31 @@ CREATE TABLE proposal_review (
 
 COMMENT ON TABLE proposal_review IS 'All Reviews.';
 COMMENT ON COLUMN proposal_review.row_id IS 'Synthetic Unique Key.';
-COMMENT ON COLUMN proposal_review.proposal_id IS 'The Proposal id this review belongs to.';
-COMMENT ON COLUMN proposal_review.user_id IS 'The user id this review belongs to.';
-COMMENT ON COLUMN proposal_review.assessor IS 'Assessors Anonymized ID.';
+COMMENT ON COLUMN proposal_review.proposal_id IS 'The Proposal this review is for.';
+COMMENT ON COLUMN proposal_review.assessor IS 'Assessors Anonymized ID';
 COMMENT ON COLUMN proposal_review.assessor_level IS 'Assessors level ID';
+COMMENT ON COLUMN proposal_review.reward_address IS 'Assessors reward address';
 
 COMMENT ON COLUMN proposal_review.impact_alignment_rating_given IS
-'The  numeric rating assigned to the proposal by the assessor.';
+'The  numeric rating assigned to the proposal by the assessor.
+DEPRECATED: Only used for Vit-SS compatibility.';
 COMMENT ON COLUMN proposal_review.impact_alignment_note IS
-'A note about why the impact rating was given.';
+'A note about why the impact rating was given.
+DEPRECATED: Only used for Vit-SS compatibility.';
+
 COMMENT ON COLUMN proposal_review.feasibility_rating_given IS
-'The numeric feasibility rating given.';
+'The numeric feasibility rating given.
+DEPRECATED: Only used for Vit-SS compatibility.';
 COMMENT ON COLUMN proposal_review.feasibility_note IS
-'A note about why the feasibility rating was given.';
+'A note about why the feasibility rating was given.
+DEPRECATED: Only used for Vit-SS compatibility.';
+
 COMMENT ON COLUMN proposal_review.auditability_rating_given IS
-'The numeric auditability rating given.';
+'The numeric auditability rating given.
+DEPRECATED: Only used for Vit-SS compatibility.';
 COMMENT ON COLUMN proposal_review.auditability_note IS
-'A note about the auditability rating given.';
+'A note about the auditability rating given.
+DEPRECATED: Only used for Vit-SS compatibility.';
 
 COMMENT ON COLUMN proposal_review.ranking IS
 'Numeric  Measure of quality of this review according to veteran community advisors.

src/event-db/migrations/V5__user.sql

@@ -17,14 +17,6 @@ CREATE TABLE catalyst_user
 CREATE UNIQUE INDEX user_catalyst_id_idx ON catalyst_user(catalyst_id);
 
 
-COMMENT ON TABLE catalyst_user IS '
-This tables stores the user account for the Review Module.
-It contains a reference to the catalyst_id that is used as public and unique identifier.
-';
-
-COMMENT ON COLUMN catalyst_user.catalyst_id IS 'The catalyst_id this account belongs to.';
-COMMENT ON COLUMN catalyst_user.enc_password IS 'The encrypted password.';
-COMMENT ON COLUMN catalyst_user.salt IS 'The salt for the password encryption.';
 COMMENT ON COLUMN catalyst_user.extra IS
 'This field is used to store all meta information about a user that 
 are required by the application. Specifically: