Suppression de compte #57 #74

brunelie · 2025-09-05T11:48:59+02:00

brunelie commented

2025-09-05 11:48:59 +02:00

Résoud #57
Un utilisateur peut supprimer son compte lorsqu'il n'a plus d'abonnement actif. Le compte Canaille est supprimé via SCIM, le compte Lago est supprimé et les factures non réglées sont toujours dans le système pour être suivies.
S'il n'y a aucune facture non réglée, le compte Stripe est aussi supprimé.
S'il y a une ou plusieurs factures non réglées, le compte Stripe n'est pas supprimé, il le sera automatiquement lorsque les paiements de toutes les factures seront marqués "successful" (ou que les factures seront void).

Résoud #57 Un utilisateur peut supprimer son compte lorsqu'il n'a plus d'abonnement actif. Le compte Canaille est supprimé via SCIM, le compte Lago est supprimé et les factures non réglées sont toujours dans le système pour être suivies. S'il n'y a aucune facture non réglée, le compte Stripe est aussi supprimé. S'il y a une ou plusieurs factures non réglées, le compte Stripe n'est pas supprimé, il le sera automatiquement lorsque les paiements de toutes les factures seront marqués "successful" (ou que les factures seront void).

brunelie added 18 commits

2025-09-05 11:48:59 +02:00

feat: adds routes and confirmation page for account deletion 97299c1588

feat: adds scim function to delete user account 0dd28daf0a

feat: user can delete their account, it deletes canaille account and logs them out 2089448c75

feat: sends an information e-mail when an account is deleted 4b3ca221d1

tests: testing account deletion and logout after account deletion 5c7d0b0194

refactor: grids are now flexbox and js menu snippet is fixed c44fd33dbc

feat: account deletion also deletes related Lago customer 90dc9bcfdd

tests: adds destroy customer in mocks de357f1800

tests: adds a test for account deletion 11ff716491

fix: edit subscruption panel does not appear anymore when user has no subscription 36ccceafb0

feat: adding Stripe API for customer deletion 6d020f3443

feat: clients with unpaid invoices are not deleted in stripe a140ab3d83

feat: deletes stripe customer on successful payment webhook message if no lago user and no invoices left for customer b9c5227180

feat: verifies invoices status after successful payments to process account deletions 4feecf8ba1

feat: prevents user from deleting account with active subscriptions 8ea93e57ac

tests: account deletion basic scenario is tested 85357bc6a1

tests: tests the prevention of account deletion if a subscription is still active ed4411e9e1

chore: fix uv.lock merge error

ci/woodpecker/push/lint Pipeline was successful

Details

ci/woodpecker/pr/lint Pipeline was successful

Details

ci/woodpecker/pr/tests Pipeline failed

Details

23a954e1e9

felix requested changes

2025-09-05 15:57:48 +02:00

tests/test_authentication.py Outdated

					
				@ -82,0 +104,4 @@

				    with patch("stripe.Customer") as stripe_customer:

				        stripe_customer.return_value = {"data": [stripe_user]}

				        res = client.get("/suppression")

				    assert ("success", "Votre compte a bien été supprimé.") in res.flashes

felix commented

2025-09-05 15:41:50 +02:00

this test should also verify that the canaille user is correctly deleted, and that the information email is correctly sent

brunelie commented

2025-09-08 09:47:43 +02:00

c'est testé ✅ 11dce1f85b

c'est testé ✅ https://forge.yaal.coop/yaal/escut/commit/11dce1f85bb5bedec3634e677c9037408799b9e7

web/app/scim.py Outdated

					
				@ -61,0 +73,4 @@

				        )

				    if user:

				        try:

				            user = scim.delete(User, username)

felix commented

2025-09-05 15:55:56 +02:00

I think this function takes user.id instead of the username

I think this function takes `user.id` instead of the username

brunelie commented

2025-09-08 08:48:35 +02:00

Dans Canaille l'id d'un utilisateur c'est son username, donc ça fonctionne avec username, mais je peux remplacer username ici par user_id pour que ça suive ce que demande SCIM et toujours lui passer l'username (le external_id en l'occurrence) quand la fonction est appelée et ça fonctionnera aussi. Ca te parait ok ?

Dans Canaille l'id d'un utilisateur c'est son `username`, donc ça fonctionne avec `username`, mais je peux remplacer `username` ici par `user_id` pour que ça suive ce que demande SCIM et toujours lui passer l'username (le `external_id` en l'occurrence) quand la fonction est appelée et ça fonctionnera aussi. Ca te parait ok ?

👍 1

brunelie commented

2025-09-08 09:48:01 +02:00

j'ai fait ça du coup d3fd8b48c8

j'ai fait ça du coup https://forge.yaal.coop/yaal/escut/commit/d3fd8b48c8a3b19090c8122f8267b06d05cbf81e

felix marked this conversation as resolved

brunelie added 2 commits

2025-09-08 09:39:24 +02:00

tests: tests that canaille user is deleter and that confirmation mail is sent 11dce1f85b

fix: scim delete account uses user_id

ci/woodpecker/push/lint Pipeline was successful

Details

ci/woodpecker/pr/lint Pipeline was successful

Details

ci/woodpecker/pr/tests Pipeline failed

Details

d3fd8b48c8

brunelie commented

2025-09-08 09:48:48 +02:00

il y a encore le pb des autres tests qui fail à cause du mock de Lago à résoudre avant de merge

ci/woodpecker/push/lint Pipeline was successful

Details

ci/woodpecker/pr/lint Pipeline was successful

Details

ci/woodpecker/pr/tests Pipeline failed

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin delete-account:delete-account

git switch delete-account

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff delete-account

git switch delete-account

git rebase main

git switch main

git merge --ff-only delete-account

git switch delete-account

git rebase main

git switch main

git merge --no-ff delete-account

git switch main

git merge --squash delete-account

git switch main

git merge --ff-only delete-account

git switch main

git merge delete-account

git push origin main

Sign in to join this conversation.

No reviewers

No labels

bug

documentation

fonctionnalité

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: yaal/escut#74

Rows
Columns