feat(InstallAppDialog): Add InstallAppDialog new component

[zatteo]

Displays a dialog with a QR code to install the flagship app

Demo : https://zatteo.github.io/cozy-ui/react/#!/SpecificDialogs

[bundlemon]

BundleMon

Unchanged files (3)

Status	Path	Size	Limits
✅	dist/cozy-ui.min.css	19.72KB	+10%
✅	transpiled/react/stylesheet.css	19.13KB	+10%
✅	dist/cozy-ui.utils.min.css	10.5KB	+10%

No change in files bundle size

Groups updated (1)

Status	Path	Size	Limits
✅	transpiled/react/**	585.46KB (+1.57KB +0.27%)	-

Final result: ✅

View report in BundleMon website ➡️

---

Current branch size history | Target branch size history

[Merkur39]

nit: On est sûr que les liens ne changent pas "souvent" ?

[zatteo]

Pas vraiment sûr. Tu proposerais quoi ? Laisser ces url ici mais mettre une props pour les override ? Ou juste via une props ?

[Merkur39]

Laisser ces url ici mais mettre une props pour les override ?

Ca me parait bien, mais pour le même besoin qu'évoqué par Crash-- ci-dessous il faudrait également pouvoir surcharger le QR Code je pense

[zatteo]

J'ai passé ça en props avec ces URLs en valeur par défaut, c'est un peu plus flexible comme ça :)

[Merkur39]

Top, mais je pensais également au QR Code, qui peut mener à différents endroit selon le contexte

[zatteo]

C'est pas faux... J'ai mis le QRCode en props également...

[Crash--]

Dans quel cas cette modale est-elle affichée ? Doit-elle tenir compte du fait qu'elle peut être dans un contexte partenaire ? Et dans ce cas, avoir des arguments supplémentaires sur l'url du playstore ?

[zatteo]

Dans quel cas cette modale est-elle affichée ? Doit-elle tenir compte du fait qu'elle peut être dans un contexte partenaire ? Et dans ce cas, avoir des arguments supplémentaires sur l'url du playstore ?

Actuellement, elle est sur mespapiers quand on veut faire un scan. Et là je dois mettre la même sur photo pour la sauvegarde.

Je sais pas trop ce qu'implique les contextes partenaire. Tu penses à quoi comme argument supplémentaire ? Des query string dans l'url ?

Sinon comme pour le commentaire d'alexis je peux proposer d'override les url via des props.

[JF-Cozy]

@zatteo est-ce que la démo est à jour ? Peux-tu mettre un lien direct plutôt que la doc globale stp ?

[JF-Cozy]

ça ne marchera pas la propagation simple des props ici. Idéalement il devrait y avoir une concaténation, et placé en plein milieu comme ici fait qu'on va override open, size, onClose, componentsProps mais pas title, content. Pour faire plus simple, on devrait l'enlever je pense.

[zatteo]

J'ai enlevé la propagation pour la simplicité. SI je l'avais mis à la fin qu'est-ce qui aurait manqué encore ?

[JF-Cozy]

le mettre à la fin fait qu'il va tout overrider, donc on perd des choses comme par exemple componentsProps={{ dialogTitle: { className: 'u-pt-2' } }}

[JF-Cozy]

je pense qu'on peut supprimer ce cas, et le remplacer par une petite phrase plus haut qui explique l'intérêt de cette modale et comment elle fonctionne.

[JF-Cozy]

autre chose dans le readme, on peut forcer le open à true si isTesting() comme ça la modale est ouverte et Argos sert à quelque chose.

[zatteo]

Génial je connaissais pas le isTesting

[cozy-bot]

🎉 This PR is included in version 88.2.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀