Skip to content

Delete all fragments which belongs to a user upon deleting #248

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

Merged
merged 9 commits into from
Apr 4, 2024

Conversation

int2001
Copy link
Contributor

@int2001 int2001 commented Apr 2, 2024

While testing on my instance i recognized a lot of old QSOs and other fragments where still in Database active.
In preparation of activating contraints somewhere in the future, we should delete everything which belongs to the User.

Affected Actions:

  • Delete User (should delete everything from the user: OQRS, Certs, QSOs, eQSLs, QSLs, bandx, stations, logbooks, etc.)
  • Delete Station-Location (should delete everything from the station: OQRS, QSOs, eQSLs, QSLs)

@int2001 int2001 marked this pull request as ready for review April 3, 2024 08:27
@HB9HIL
Copy link
Contributor

HB9HIL commented Apr 3, 2024

Works great :)

@phl0
Copy link
Contributor

phl0 commented Apr 4, 2024

Tested with the following result:

  • Emptying log table works
  • Emptying banduxer table works
  • Emptying user_options works
  • Emptying lotw_certs works
  • Emptying station_profile works

Not working:

  • Emptying eQSL_images
  • Emptying qsl_images

@phl0
Copy link
Contributor

phl0 commented Apr 4, 2024

Confirmed. (e)qsl_images tables are now emptied as well.

@phl0
Copy link
Contributor

phl0 commented Apr 4, 2024

Rest also tested. Deleting station_location removes connected data.
Removing single QSOs also removes relevant stuff from DB and file system.
Good to go.

@int2001 int2001 merged commit 806ba71 into wavelog:dev Apr 4, 2024
@int2001 int2001 deleted the delete_really branch April 15, 2024 12:56
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.

3 participants