Skip to content

issue-137-User-Deletion-feature#422

Open
Saishiva3000 wants to merge 1 commit into
FindFirst-Development:mainfrom
Saishiva3000:issue-137-userDeletion-feature
Open

issue-137-User-Deletion-feature#422
Saishiva3000 wants to merge 1 commit into
FindFirst-Development:mainfrom
Saishiva3000:issue-137-userDeletion-feature

Conversation

@Saishiva3000

Copy link
Copy Markdown

Issue number: resolves #137

              ## What is the current behavior ?  ---->  User cannot delete account



              ## What is the new behavior? ------> added a end point to delete user in **UserController** and added integration   
                                                                             tests in  **UserControllerTest**>

Note:

the thing i cannot run the application on my laptop because of time zone error of postgres and i can't possibly run i don't know why , i can't solve that issue

please run tests before merging my pleased request (test is correct)

@R-Sandor

Copy link
Copy Markdown
Collaborator

What is this timezone error? This may be a bigger region/date format issue that I didn't consider, i.e. people in X country represent DateTime differently or is it just a version of postgres container that doesn't work with arm or something similar?

@Saishiva3000

Copy link
Copy Markdown
Author

the issue i got is that the postgres container could not start because the timezone error , i asked ai about this error it said something like the jvm is configuring timezone as India/calcutta which postgres won't support.

check out the code once , there are already some methods written in bookmark and tag service which i used to delete user bookmarks and tags.

another issue is about the tokens , did you make the cascade delete for tokens like when user deletes all the tokens also delete or they expire on ttl logic and get deleted

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.

User can delete account

2 participants