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

👋 Say goodbye to intents #219

Open
gregorylegarec opened this issue Oct 26, 2017 · 9 comments
Open

👋 Say goodbye to intents #219

gregorylegarec opened this issue Oct 26, 2017 · 9 comments
Assignees

Comments

@gregorylegarec
Copy link
Contributor

Nous allons être amenés dans nos prochain sprints à implémenter quelques évolutions au sein des intents. Comme le code est déjà dupliqué entre cozy-bar et cozy-client et que nous nous posons la question de l'extraire de cozy-client de longue date, nous pensons que le moment est venu.

L'idée serait je pense de proposer les intents au sein d'une lib indépendante, que nous appellerions soit cozy-intents ou soit cozy-interapp, cette dernière ayant ma préférence puisqu'il est probable que nous ajoutions d'autres fonctionnalités autres que les intents et liées à l'inter-app.

Dernier point, et j'aimerais avoir votre avis, nous pouvons créer un nouveau repo et ou bien intégrer la lib à cozy-drive et son mono-repo, nous avons par contre pour cela besoin d'être capables de la tester et la publier.

Commentaires et remarques bienvenus.

@goldoraf
Copy link
Contributor

💯 pour en faire une lib indépendante de cozy-client(-js)

Et oui, je pense que ce serait mieux de l'intégrer au monorepo ;)

@ptbrowne
Copy link
Contributor

Quid de la rétro-comptatibilité avec cozy.client.intents ?

Ok pour cozy-interapp.

@enguerran
Copy link
Contributor

Bonne idée pour l'extraction de la bibliothèque.
Aucun avis sur le nommage tant que le nom est expressif quant à l'intention de la bibliothèque.
Je ne suis pas sûr que le monorepo github soit une bonne idée pour tout ce qui est gestion de projets (issues, PR, wiki), adieu les workflows git.

@y-lohse
Copy link
Contributor

y-lohse commented Oct 26, 2017

Vous avez ma bénédiction. Préférence aussi pour cozy-interapp, comme certains comportements vont surement diverger un peu d'un système d'intents classique.

Je ne pense pas que ca pose techniquement problème de tester et publier une lib à l'intérieur du mono-repo. Par contre ca à surtout un intêret si les projets qui l'utilisent beaucoup sont eux aussi dans le mono-repo. Comme a priori ce n'est pas le cas, je pense qu'un dépot à part a plus de sens.

@gregorylegarec
Copy link
Contributor Author

@ptbrowne Bonne remarque, je pense extraire la lib mais laisser encore la version actuelle dans cozy-client-js le temps que les apps migrent. Dans un premier temps seul les apps nécessitant les dernières évolutions devront switcher vers la nouvelle lib.

@kosssi
Copy link
Contributor

kosssi commented Oct 26, 2017

Je n'arrive plus du tout a vous suivre :O

On part sur une organisation monorepo ou pas ? ... enfin bon faite comme vous le sentez mais pour moi c'est dans le monorepo

@gregorylegarec
Copy link
Contributor Author

@kosssi J'allais dire que le seul inconvénient pour le moment est que ni Drive ni Photos n'utilisent les intents. MAIS cela risque de changer très prochainement au vu de ce que nous sommes en train d'implémenter dans Collect.

@enguerran
Copy link
Contributor

@kosssi a écrit :

On part sur une organisation monorepo ou pas ?

cf. https://mattermost.cozycloud.cc/test-team/pl/5tj9inum1bdwidb6wbp13dqy5o

@clochix
Copy link
Contributor

clochix commented Oct 30, 2017

Bonjour,

Merci de penser à la documentation de développement, y compris y annoncer rapidement que le code des intentions risque d’évoluer.

@enguerran enguerran removed their assignment Jun 3, 2021
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

No branches or pull requests