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

[TECH] Mettre à jour Node et les paquets Pix Site (PIX-1633). #220

Merged
merged 33 commits into from
Nov 20, 2020

Conversation

sbedeau
Copy link
Contributor

@sbedeau sbedeau commented Nov 18, 2020

🦄 Problème

Nous utilisons désormais Node 14 et non plus 12.

Les paquets de Pix Site ne sont pas à jour.
Screenshot 2020-11-18 at 13 28 34

🤖 Solution

  • Mettre les paquets à jour unitairement avec exécution des tests
  • Mettre à jour node

🌈 Remarques

💯 Pour tester

Se balader sur les sites

@sbedeau sbedeau added 🚧 Development in progress team-evaluation PR relatives à l'expérience d'évaluation cross-team Toutes les équipes de dev labels Nov 18, 2020
@sbedeau sbedeau self-assigned this Nov 18, 2020
@sbedeau sbedeau marked this pull request as ready for review November 18, 2020 18:40
@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@sbedeau sbedeau changed the title [TECH] Mettre à jour les paquets Pix Site. [TECH] Mettre à jour Node et les paquets Pix Site. Nov 18, 2020
@sbedeau sbedeau changed the title [TECH] Mettre à jour Node et les paquets Pix Site. [TECH] Mettre à jour Node et les paquets Pix Site (PIX-1633). Nov 19, 2020
@@ -1,7 +1,6 @@
import { transports } from 'winston'

export default {
mode: 'universal',
Copy link
Member

Choose a reason for hiding this comment

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

Pourquoi on retire ce mode ? Est-ce le mode par défaut ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Au moment du build, un warning indique que c'est une option dépréciée

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Après vérification, c'est bien le mode par défaut https://nuxtjs.org/docs/2.x/configuration-glossary/configuration-mode

J'avoue avoir suivi naïvement le conseil du build nuxt qui disait que l'on pouvait remove safely

@@ -105,7 +105,7 @@ export default {
this.$modal.show('videoModal')
Copy link
Member

Choose a reason for hiding this comment

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

Je pense qu'il faut revoir la modal avec la montée de version de https://github.com/euvl/vue-js-modal#readme

Sur la home page, le bouton pour lancer la vidéo ne fonctionne plus.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bien vu, merci.

Compte tenu du message suivant, je vais roll-back le commit dédié

Version 2.0.0 release candidates will have breaking changes until it is 2.0.1. If you need a more stable version, please use 1.3.34.

@sbedeau sbedeau requested a review from MelanieMEB November 19, 2020 09:33
@sbedeau sbedeau merged commit b87240c into dev Nov 20, 2020
@sbedeau sbedeau deleted the tech-bump-packages branch November 20, 2020 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants