[Bug] Ad Image Incomplete src URL

[ferrariofilippo]

[Bug] Ad Image Incomplete src URL

[yassindeb]

Yeah you have a point here, alright then we shall do as you did with the ad creation section: using relative paths.

[ferrariofilippo]

I'm not sure we should save the full path: it depends on too many variables (https://INSTANCE_NAME.supabase.co/storage/VERSION/object/BUCKET_TYPE/BUCKET_NAME).
I think we should rather use supabase.storage.from("BUCKET_NAME").getPublicUrl(SAVED_URL).data.publicUrl to get images.

[yassindeb]

The thing is that the uploaded avatars appear in the db with a src column that contains the full image URL while, the ad pictures don't it's not that big of a deal, we just have to decide whether to have 2 separate approaches or one.
In my opinion it makes sense to have the full url in case we want to migrate to our own database istance in the future.

[ferrariofilippo]

In my opinion it makes sense to have the full url in case we want to migrate to our own database istance in the future.

Actually, in this case it would be better to have the relative paths, otherwise we would need to update all the rows on the new DB.
We might choose to hard-code the first part of the path, instead of retrieving it using the Supabase library, but I still think that the relative URLs would be better