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

[BUGFIX] Télécharger tous les assets disponibles #674

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented Jun 21, 2024

🦄 Problème

Lorsque des params sont passés au serveur h3, le serveur ne veut pas les gérer et l'url est skippé ce qui fait que l'asset n'est pas téléchargé

🤖 Proposition

Sur notre fork nous retirons les params envoyé au h3 et par conséquent à Prismic cela peut donc avoir des impacts sur les tailles d'images demandés 1024pix/nuxt-image@7f6025d

Le vrai correctif aurait été de faire en sorte que le h3 accepte et transfère les params à voir plus tard

🌈 Remarques

💯 Pour tester

@VincentHardouin VincentHardouin changed the title fix: use last version of nuxt-image-prismic-fix to handle all assets [BUGFIX] Télécharger tous les assets disponibles Jun 21, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr674/environment

Copy link
Member

@yannbertrand yannbertrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK Tech ✅

@yannbertrand
Copy link
Member

Plus de skipped dans les logs Scalingo.

@pix-service-auto-merge pix-service-auto-merge merged commit 14be64b into dev Jun 21, 2024
5 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the fix-download-assets branch June 21, 2024 09:59
@yannbertrand
Copy link
Member

Il n'y a plus d'assets téléchargées en dehors de Scalingo ✅

On passe à 3.3MB téléchargés sur la home.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants