linagora / tmail-backend

GNU Affero General Public License v3.0
30 stars 17 forks source link

User deletion: PublicAsset #1043

Closed quantranhong1999 closed 1 month ago

quantranhong1999 commented 1 month ago

Why

Upon user deletion, related PublicAsset(s) should be cleaned.

Upon username change, related PublicAsset(s) should be migrated.

How

DoD

Unit tests + integration tests (for guice binding) Notice to Guice binding for postgres-app as well.

Arsnael commented 1 month ago

Nice catch, was missing

quantranhong1999 commented 1 month ago

Skip username change migration for public asset cf https://github.com/linagora/james-project-private/pull/810#discussion_r1607978710

hungphan227 commented 1 month ago

pr https://github.com/linagora/tmail-backend/pull/1065