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

feat: Allow users to delete their own accounts #8548

Merged
merged 4 commits into from
Jun 15, 2023
Merged

Conversation

john-gom
Copy link
Contributor

@john-gom john-gom commented Jun 12, 2023

What

Enable account deletion for regular users with suitable warnings

Screenshot

image

image

Related issue(s) and discussion

@john-gom john-gom requested a review from a team as a code owner June 12, 2023 16:14
@github-actions github-actions bot added Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. Translations We use a non-standard version of GetText, lack language variants support translate.openfoodfacts.org 👥 Users labels Jun 12, 2023
@stephanegigandet
Copy link
Contributor

Looks good, thank you!

From @alexgarel on Slack:

"I would really have liked to have a separate DELETE button if possible, the delete checkbox with "save" is a bit difficult to understand I think !
But I know it might be difficult because of the way the code is written"

@stephanegigandet
Copy link
Contributor

Having a separate Delete button should be relatively easy to add through the template I think.

@john-gom
Copy link
Contributor Author

Having a separate Delete button should be relatively easy to add through the template I think.

I've just pushed a change to do this

@codecov-commenter
Copy link

codecov-commenter commented Jun 13, 2023

Codecov Report

Merging #8548 (6460fc8) into main (2dee723) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #8548   +/-   ##
=======================================
  Coverage   48.70%   48.70%           
=======================================
  Files         114      114           
  Lines       21412    21412           
  Branches     4789     4789           
=======================================
  Hits        10429    10429           
  Misses       9698     9698           
  Partials     1285     1285           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@stephanegigandet
Copy link
Contributor

I've just pushed a change to do this

Thanks @john-gom

Maybe we could put the "Danger zone" after the regular "Save" button:

image

(maybe with an extra
or 2 or a css style to separate well the danger zone from the rest of the form).

What do you think?

@john-gom
Copy link
Contributor Author

john-gom commented Jun 14, 2023

I've just pushed a change to do this

Thanks @john-gom

Maybe we could put the "Danger zone" after the regular "Save" button:

image

(maybe with an extra or 2 or a css style to separate well the danger zone from the rest of the form).

What do you think?

Hi @stephanegigandet

Very easy to do but when I look at it I'm a bit concerned as the "call to action" for a form is typically at the bottom so I think having delete there could result in it being more likely to be clicked.

Cosmetically I also prefer having the group boxes together rather than having the save button in-between them

@sonarcloud
Copy link

sonarcloud bot commented Jun 14, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Thank you very much!

@stephanegigandet stephanegigandet merged commit 6a294d0 into main Jun 15, 2023
15 checks passed
@stephanegigandet stephanegigandet deleted the issue/6435 branch June 15, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. Translations We use a non-standard version of GetText, lack language variants support translate.openfoodfacts.org 👥 Users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow to initiate account deletion from the website/API
3 participants