Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

[FEATURE] Profile Delete feature #8260

Closed
amitthecoder opened this issue Jul 19, 2023 · 25 comments
Closed

[FEATURE] Profile Delete feature #8260

amitthecoder opened this issue Jul 19, 2023 · 25 comments
Labels
💻 aspect: code undefined ⭐ goal: addition undefined 🔢 points: 5 undefined 🔒 staff only undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)

Comments

@amitthecoder
Copy link

Description

Hello LinkFree Contributors, Today I am creating a LnkeFree account For some reason I want to delete my account but delete option is not available. I removed github authorizations but my profile shows on LinkFree.

Screenshots

No response

Additional information

No response

@amitthecoder amitthecoder added ⭐ goal: addition undefined 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) labels Jul 19, 2023
@github-actions
Copy link
Contributor

To reduce notifications, issues are locked until they are https://github.com/EddieHubCommunity/LinkFree/labels/%F0%9F%8F%81%20status%3A%20ready%20for%20dev and to be assigned. You can learn more in our contributing guide https://github.com/EddieHubCommunity/LinkFree/blob/main/CONTRIBUTING.md

@github-actions github-actions bot locked and limited conversation to collaborators Jul 19, 2023
@eddiejaoude eddiejaoude added 💻 aspect: code undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned) 🔢 points: 5 undefined and removed 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) labels Jul 19, 2023
@github-actions github-actions bot unlocked this conversation Jul 19, 2023
@github-actions
Copy link
Contributor

The issue has been unlocked and is now ready for dev. If you would like to work on this issue, you can comment to have it assigned to you. You can learn more in our contributing guide https://github.com/EddieHubCommunity/LinkFree/blob/main/CONTRIBUTING.md

@eddiejaoude
Copy link
Member

This was coming soon, but we can make it a higher priority.

Can I ask why you want to delete your account, not sure what you mean by "some reason"?

@abhijeetrayy
Copy link

I can do that but need your views on it to procced or not (people/geeks)

@amitthecoder
Copy link
Author

amitthecoder commented Jul 19, 2023 via email

@amitthecoder
Copy link
Author

amitthecoder commented Jul 19, 2023 via email

@eddiejaoude
Copy link
Member

Because I am already using bio.link and when I created LinkFree account I realized the bio.link is better, So I want to delete LinkFree account..

Could you be more specific @amitthecoder , what is better about bio link so we can make improvements? Thanks

@eddiejaoude eddiejaoude changed the title [FEATURE] Add Profile Delete feature [FEATURE]Profile Delete feature Jul 20, 2023
@eddiejaoude eddiejaoude changed the title [FEATURE]Profile Delete feature [FEATURE] Profile Delete feature Jul 20, 2023
@eddiejaoude
Copy link
Member

eddiejaoude commented Jul 20, 2023

I can do that but need your views on it to procced or not (people/geeks)

I suggest adding the delete button to the profile edit page, similar to how we have the delete button for events

DELETE EVENTS BUTTON

Screenshot 2023-07-20 at 09 15 26

EDIT PROFILE - delete button can go here next to the save

Screenshot 2023-07-20 at 09 15 14

Use the same process with the confirmation dialogue...

Screenshot 2023-07-20 at 09 15 34

Then on confirmation, change the profile isEnabled to false - we will add a cron later to delete all the data, as we will have lots of stats to delete etc

Does that make sense @abhijeetrayy ?

@webAbhi
Copy link

webAbhi commented Jul 20, 2023

Hi @eddiejaoude if you don't mind can you assign this issue to me ? this look really interesting, i would love to work on this.

@yoyoekta
Copy link
Contributor

Screenshot 2023-07-20 at 09 15 34

We can also add confirmation like, the user need to type in the username to permanently delete profile, to add extra layer of safety?

@vishalsingh2972
Copy link

can I work on this if the issue is still open?

@eddiejaoude
Copy link
Member

@webAbhi you asked first, so I have assigned it to you

@webAbhi
Copy link

webAbhi commented Jul 23, 2023

@eddiejaoude i think isEnabled flag is not properly handled in backend, when we send isEnabled: false as payload then in response we will receive the value as true, which is weird. i am using the same PUT api which is been used for profile data update. even tried replicating the same thing from postman but getting the same result. BTW i have completed the code for this.

@eddiejaoude
Copy link
Member

@eddiejaoude i think isEnabled flag is not properly handled in backend, when we send isEnabled: false as payload then in response we will receive the value as true, which is weird. i am using the same PUT api which is been used for profile data update. even tried replicating the same thing from postman but getting the same result. BTW i have completed the code for this.

Yes you will need to update the API code for the PUT to accept this field and update it.

Also, if an account is disabled, they will need the functionality to enable it also

@webAbhi
Copy link

webAbhi commented Jul 23, 2023

@eddiejaoude i think isEnabled flag is not properly handled in backend, when we send isEnabled: false as payload then in response we will receive the value as true, which is weird. i am using the same PUT api which is been used for profile data update. even tried replicating the same thing from postman but getting the same result. BTW i have completed the code for this.

Yes you will need to update the API code for the PUT to accept this field and update it.

Also, if an account is disabled, they will need the functionality to enable it also

So are we looking to make this API update anytime soon or we need to wait for next deployment ? regarding functionality of enabling a disable profile, you can attach the screenshot where you want to see that. i can add that too in same PR.

@eddiejaoude
Copy link
Member

Yes this should be part of this pull request 👍 it does not make sense to be in a separate PR

@webAbhi
Copy link

webAbhi commented Jul 31, 2023

Hey @eddiejaoude, is there any updates on API ?

@eddiejaoude
Copy link
Member

@webAbhi what do you mean? I thought you were making the required changes in your pull request?

@webAbhi
Copy link

webAbhi commented Jul 31, 2023

@eddiejaoude okay my bad, but where are we planning to add a button to enable a disabled profile ? How about this page 👇👇?

Screenshot 2023-07-31 at 6 44 28 PM

@eddiejaoude
Copy link
Member

eddiejaoude commented Aug 1, 2023

@webAbhi I gave an example in a comment 2 weeks ago further up at the beginning of this conversation - here is the directly link to that reply #8260 (comment)

@SaraJaoude
Copy link
Member

Due to lack of activity on #8437 I am unassigning and marking this with a staff label as it now has greater priority.

@shivamsethi
Copy link

@eddiejaoude - I can work on this one. Can you assign this to me?

@eddiejaoude
Copy link
Member

eddiejaoude commented Oct 1, 2023

@shivamsethi please don't tag individual maintainers when it is not relevant (as mentioned in our contributing guide), other maintainers can assist also.

Please see the issue label, it is marked as staff only, so only maintainers can work on this issue - you can learn more about our labels in our docs.

@shivamsethi
Copy link

Alright Sure :) Thank you :)

@eddiejaoude
Copy link
Member

eddiejaoude commented Nov 22, 2023

Closing as superseded with issue #9821

ℹ️ eddiejaoude has some opened assigned issues: 🔧View assigned issues

@eddiejaoude eddiejaoude closed this as not planned Won't fix, can't repro, duplicate, stale Nov 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code undefined ⭐ goal: addition undefined 🔢 points: 5 undefined 🔒 staff only undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants