enBloc-org / kindly

Open source repository for the development of Kindly
GNU General Public License v3.0
18 stars 21 forks source link

Limit Profile pictures to 1 per profile #252

Closed camelPhonso closed 2 months ago

camelPhonso commented 5 months ago

Expected Behaviour

When a user edits their profile image the old image should be deleted to avoid overloading our storage buckets.

Current Behaviour

When a User edits their profile to change the profile picture the old image is kept in our storage bucket even though noting in the database points to it anymore. As a part of the editing process that picture should be removed when the new one is uploaded.

Steps to Reproduce

  1. Log into Kindly
  2. Visit your profile page
  3. Edit the profile and add a new profile picture
  4. Run npx supabase status on your terminal and visit the Studio URL link
  5. In your supabase console navigate to storage > images and you can still find your old profile image there
github-actions[bot] commented 3 months ago

🤖 meep morp!

This Issue is now marked as stale because there has been no activity for a while.

🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days.

💡 To unstale this Issue please push any commits that are ready or provide an update in the comments.

camelPhonso commented 3 months ago

this isn't stale, bot! This PR is in active reviews

github-actions[bot] commented 3 months ago

🤖 meep morp!

This Issue is now marked as stale because there has been no activity for a while.

🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days.

💡 To unstale this Issue please push any commits that are ready or provide an update in the comments.

github-actions[bot] commented 2 months ago

🤖 meep morp!

This Issue is now marked as stale because there has been no activity for a while.

🔎 The Issue will be automaticaly unassigned and moved to the Backlog if it doesn't receive new activity in the next 7 days.

💡 To unstale this Issue please push any commits that are ready or provide an update in the comments.