Skip to content
This repository has been archived by the owner on Feb 28, 2023. It is now read-only.

Updated vuejs to v.2.5.23 + storybook 5.0.6 #864

Merged
merged 10 commits into from
Apr 10, 2019

Conversation

chuckmah
Copy link
Contributor

@chuckmah chuckmah commented Apr 8, 2019

@ulaval/modul-components

PR Checklist

  • Provide a small description of the changes introduced by this PR

Mise a jour de storybook et vue au derniere version mineur supporté

La mise a jour de vue a nécessité des changements au niveau du typing typescript. J'ai du ajouter des cast a plusieurs endroit pour enlever les erreurs de compilations.

  • Include links to issues
  • Openshift deployment requested

@chuckmah chuckmah added this to the 1.0.0-beta.92 milestone Apr 8, 2019
@chuckmah chuckmah changed the title mise a jour vue Updated vuejs minor version+ storybook Apr 8, 2019
@chuckmah chuckmah mentioned this pull request Apr 8, 2019
3 tasks
chuckmah and others added 2 commits April 9, 2019 09:03
* Changer la commande d'exécution de node

* Ajouter une commande pour rouler storybook sur openshift

* Ajouter le paramètre --port à storybook:openshift

* mise a jour ts-jest
src/directives/badge/badge.ts Outdated Show resolved Hide resolved
src/mixins/element-queries/element-queries.ts Show resolved Hide resolved
src/directives/sortable/sortable.ts Outdated Show resolved Hide resolved
@chuckmah chuckmah changed the title Updated vuejs minor version+ storybook Updated vuejs to v.2.5.23 + storybook 5.0.6 Apr 9, 2019
@@ -79,10 +80,13 @@
"@types/storybook__addon-storyshots": "^3.4.8",
"@types/storybook__vue": "^5.0.0",
"@types/webpack-env": "^1.13.6",
"@vue/test-utils": "^1.0.0-beta.28",
"@vue/test-utils": "^1.0.0-beta.29",
Copy link
Contributor

Choose a reason for hiding this comment

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

Pour ma culture personnelle: Est-ce que finalement ça aidé de passer à vue-test-utils 29?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui méga duper breaking change, important a savoir ->

vuejs/vue-test-utils#1137

je vais en parler a la prochaine rencontre UI

Copy link
Member

Choose a reason for hiding this comment

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

WoW! C'est vraiment intéressant de le savoir! Nous allons justement mettre à jour vue test utils demain à la version 29. C'est un excellent timing!

this.calendarState.buildCurrentCalendar()
);
if (this.$scopedSlots) {
return (this.$scopedSlots as any).default(
Copy link
Contributor

Choose a reason for hiding this comment

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

C'est quoi le type de base des slots ici?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

aucune idee les type on changer dans vue >2.5.18 jai mit ca pour satisfaire la compil si tu as une meilleur idee je suis ouvert au suggestion 👍

@chuckmah chuckmah modified the milestones: 1.0.0-beta.92, 1.0.0-beta.93 Apr 10, 2019
@chuckmah chuckmah merged commit 64ae124 into develop Apr 10, 2019
@chuckmah chuckmah deleted the echancement/mise_a_jour_vue_storybook branch April 10, 2019 19:08
@chuckmah chuckmah restored the echancement/mise_a_jour_vue_storybook branch April 11, 2019 13:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants