ulaval / modul

Library of VueJS components.
Apache License 2.0
10 stars 3 forks source link

[m-dialog] Déplacer l'enregistrement du service $dialog #323

Closed setur52 closed 4 years ago

setur52 commented 4 years ago

Description

Pour pouvoir wrapper le service $dialog dans Brio on doit initialiser le plugin dans un fichier séparé comme pour le toast

Types de changements

Comment cela peut-il être testé?

Inclure cette section dans les release notes

Liens internes

jfdion commented 4 years ago

@chuckmah on a le même besoin que pour le toast dans Brio. On veut pouvoir réutiliser le service de dialog de modul sans enregistrer le plugin de dialog.

Il faudrait avoir une stratégie pour permettre cette évolution et marquer l'ancienne méthode comme deprecated.

Je ne sais pas si cette façon fonctionnerait pour aller vers une transition permanente

- dialog 
  - dialog-service.ts // v 1.x deprecated - l'enregistrement du pluging de cette façon sera retiré au passage de la version 2.0 de modul
  - dialog-service // v 2.0 et plus
    - dialog-service.plugin.ts
    - dialog-service.ts 

le fichier dialog-service.ts importe le fichier de plugin de la solution permanente. Comme ça on a un seul tree de fonctions à maintenir

chuckmah commented 4 years ago

@jfdion @setur52

oui le changement est ok juste a laisser le default export du plugin dans le fichier dialog-service.ts

du genre

// @depracated sera enlevé dans la 2.0
export { DialogServicePlugin as default } from './dialog-service-plugin';
setur52 commented 4 years ago

@chuckmah c'est fait