Skip to content
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

Stockage des médias dans Appwrite Storage au lieu de l'application #37

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

luclu7
Copy link
Member

@luclu7 luclu7 commented Mar 6, 2025

C'est un peu une moyenne-grosse PR, mais promis, c'est pas trop compliqué.

Actuellement, les médias (les avatars des artistes) sont stockés dans web/src/assets/avatars, ce qui n'est pas terrible. Il a donc été décidé de migrer les assets dans le stockage proposé par Appwrite, pour être intégré avec le reste de l'application.

Présentement, dans la table des standistes, il y a une entrée « image » qui contient le nom du fichier de leur avatar. On va alors réutiliser ce champ pour stocker l'identifiant de l'image dans Appwrite Storage.

Côté scripts

Création d'un fichier helper pour gérer l'upload et la suppression des avatars, lors de la création/suppression des utilisateurs.

Côté web

Passage du petit bricolage avec import.meta.glob("/src/assets/avatars/*.jpg" à un composant dédié qui affiche l'avatar de l'artiste, avec alt avec son nom, en fonction de son id et de son nom.

Resolves #5

luclu7 added 4 commits March 5, 2025 11:40
- Add react-error-boundary for robust image loading
- Create ArtistImage component with suspense and error handling
- Implement image preloading in PWA service worker
- Add runtime caching for Appwrite storage assets
- Refactor image loading logic to use Appwrite storage
@luclu7 luclu7 added the enhancement New feature or request label Mar 6, 2025
@luclu7 luclu7 added this to the Japan Expo 2025 milestone Mar 6, 2025
@luclu7 luclu7 self-assigned this Mar 6, 2025
Copy link

netlify bot commented Mar 6, 2025

Deploy Preview for stamp-rally-devel ready!

Name Link
🔨 Latest commit 7d9d78a
🔍 Latest deploy log https://app.netlify.com/sites/stamp-rally-devel/deploys/67c97de9dcb70d00083b48f5
😎 Deploy Preview https://deploy-preview-37--stamp-rally-devel.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

cloudflare-workers-and-pages bot commented Mar 6, 2025

Deploying sedeto-vtube-stamp-rally with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7d9d78a
Status: ✅  Deploy successful!
Preview URL: https://a68a587a.sedeto-vtube-stamp-rally.pages.dev
Branch Preview URL: https://5-external-medias.sedeto-vtube-stamp-rally.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Supprimer les médias (avatars, logo) du repo
1 participant