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

Mise à jour Phoenix LiveView de 0.3.0 à 0.14.0 #1431

Merged
merged 30 commits into from
Jan 7, 2021
Merged

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Dec 23, 2020

Relatif à #1423 ; je commence une spike pour mettre à jour cette partie qui vu le changelog nécessite quelques itérations.

Tâches à prévoir

  • Documenter le processus d'upgrade de la partie cliente de LiveView (via yarn) - EDIT: j'ai ajouté un test qui plantera si on n'est pas à jour, car c'est galère à débugger potentiellement, et simple à vérifier
  • Ajouter un ou deux tests (qui auraient détecté la casse en passage à 0.5.x) - je pense que ça peut attendre plus tard (autre PR de mise à jour à 0.15.x future)
  • Upgrader jusqu'à 0.5.2
  • Upgrader la suite: je n'ai pu aller que jusqu'à 0.14.0 pour l'instant car après il faudra mettre à jour Phoenix également

Notes d'upgrade

  • Je procède à la MAJ pas par grosse nécessité de feature, mais parce que ça bloque la mise à jour de Phoenix lui-même et aussi l'installation d'autres librairies
  • LiveView est utilisé à deux endroits dans le backoffice : complétion sur les communes, et bouton "valider", j'ai vérifié que ça marchait petit à petit
  • Le changelog de LiveView est disponible ici
  • Il ne suffit pas de mettre à jour la partie dépendance Hex, il faut également mettre à jour la partie javascript à l'aide de: cd cd apps/transport/client && yarn upgrade phoenix_live_view (vérifier dans yarn.lock la version de phoenix_live_view)

@thbar thbar changed the title [WIP] Mise à jour Phoenix Live View Mise à jour Phoenix LiveView de 0.3.0 à 0.14.0 Jan 5, 2021
@thbar thbar marked this pull request as ready for review January 5, 2021 17:28
@thbar thbar requested a review from fchabouis January 5, 2021 17:29
@thbar
Copy link
Contributor Author

thbar commented Jan 6, 2021

@antoine-de je n'arrive pas à t'ajouter en reviewer, c'est curieux !

@antoine-de
Copy link
Member

étrange 🤔

@thbar
Copy link
Contributor Author

thbar commented Jan 7, 2021

J'ai ajouté un petit check pour nous prévenir des upgrades manquantes de la partie javascript, et je déploie.

@thbar thbar merged commit ac1d564 into master Jan 7, 2021
@thbar thbar deleted the upgrade-phoenix branch January 7, 2021 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants