Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add delete account API #127

Merged
merged 20 commits into from
Dec 12, 2024
Merged

Add delete account API #127

merged 20 commits into from
Dec 12, 2024

Conversation

e32wong
Copy link
Contributor

@e32wong e32wong commented Dec 12, 2024

Changes:

  • Added delete account API
    • User accounts that had been deleted will be marked with a boolean flag isDeleted in the database, and all of the posts and comments will be marked for deletion
    • New users will be assigned with a UUID as the username until they complete the onboarding steps
  • Removal of custom Pinia persistent plugin to resolve app state and synchronization bugs
  • Removed session store and clear the web crypto store whenever user logs out

@e32wong e32wong changed the title Delete account API Add delete account API Dec 12, 2024
@e32wong e32wong marked this pull request as ready for review December 12, 2024 03:28
);
--> statement-breakpoint
CREATE TABLE IF NOT EXISTS "user" (
"id" uuid PRIMARY KEY NOT NULL,
"organisation_id" integer,
"configured_user_name" boolean DEFAULT false NOT NULL,
"user_name" varchar(23) NOT NULL,
"user_name" varchar(36) NOT NULL,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename to username

"created_at" timestamp (0) DEFAULT now() NOT NULL,
CONSTRAINT "user_unique_topic" UNIQUE("user_id","post_topic_options")
CONSTRAINT "user_unique_topic" UNIQUE("user_id","post_tag_id")
);
--> statement-breakpoint
CREATE TABLE IF NOT EXISTS "user" (
"id" uuid PRIMARY KEY NOT NULL,
"organisation_id" integer,
"configured_user_name" boolean DEFAULT false NOT NULL,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename to configured_username

....etc

@@ -69,6 +69,300 @@ export const MIN_LENGTH_USERNAME = 3;

export const PEPPER_VERSION = 0;

export const POST_TAGS: Record<string, string> = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename to TOPICS

@nicobao nicobao merged commit bdf0c48 into zkorum:main Dec 12, 2024
1 check passed
@nicobao
Copy link
Member

nicobao commented Dec 18, 2024

Relates to #104

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants