prjseal / Umbraco-Community-Quiz

Source code for the Community Quiz website
https://communityquiz.net
Other
8 stars 16 forks source link

Build delete profile functionality #21

Closed prjseal closed 2 years ago

prjseal commented 2 years ago

Add a page to the member dashboard where the user can tick a box and press a button to delete their profile. This will delete everything about them from the system including their profile image, quiz results and notifications but not questions they have set.

TECKSPEED commented 2 years ago

Paul, if you don't mind I will pick this one up, among some others as time sees fit.

prjseal commented 2 years ago

Yeah go for it. Thanks mate. Please keep them in separate branches.

TECKSPEED commented 2 years ago

@prjseal almost finished up with this, running into an issue with deleting QuizResults for some odd reason. Will update if I get completely stuck...stay tuned

prjseal commented 2 years ago

Great to hear. Looking forward to seeing it 🤞🏻

TECKSPEED commented 2 years ago

@prjseal One quick question for you, using sql lite nothing special needs configuring correct? For some odd reason deleting is causing a table lock regardless of the method in which you try to delete when using the NPoco Object.

There is no specific app setting or configuration value I need set correct?

For reference: SQLite Error 6: 'database table is locked: QuizResult

prjseal commented 2 years ago

This might be something we don't need to worry about when it is live. SQLite only allows one transaction at a time. It could be that constraint getting in the way.

If you think the logic is sound I'm happy for you to submit your PR and I can test it on Big SQL

TECKSPEED commented 2 years ago

Ah, understood, yeah I think the logic is sound, but was just trying to proof the changes before submitting the PR.

Let me submit the PR and we can address any issues (hopefully none as they pop up)

prjseal commented 2 years ago

Fixed in #43