Use assistant (search and AI) from cozy-dataproxy-lib #3266
Conversation
| {isPublic ? ( | ||
| children | ||
| ) : ( | ||
| <DataProxyProvider>{children}</DataProxyProvider> |
There was a problem hiding this comment.
je crois qu'on gère les providers public autrement 🤔
DriveProvider est partagé entre les différentes targets, mais ensuite chaque target a ses propres provider, il me semble
There was a problem hiding this comment.
Je suis pas hyper fan de cette solution mais j'ai pas trouvé de meilleur endroit. En regardant le code la seule différence que j'ai vu entre public et non public c'est la prop isPublic pour la target public <App isPublic ...></App>
| @@ -15,5 +15,12 @@ jest.mock('cozy-intent', () => ({ | |||
| useWebviewIntent: jest.fn() | |||
| })) | |||
|
|
|||
| jest.mock('cozy-dataproxy-lib', () => ({ | |||
There was a problem hiding this comment.
pourquoi fait-on ça ? Un commentaire serait bienvenu je pense. Est-ce qu'on devrait généraliser l'approche à toutes les libs ?
There was a problem hiding this comment.
Deux raisons :
- les tests échouent avec l'import de cozy-dataproxy-lib avec des sombres histoires de babel pas à jour (et je n'ai pas trop envie d'investiguer ça maintenant)
- il n'y a aucune raison de tester ces composants dans les apps qui sont autonomes
Pas sûr que ce soit généralisable.
BundleMonFiles updated (6)
Unchanged files (12)
Total files change +470.95KB +9.53% Groups updated (4)
Unchanged groups (3)
Final result: ✅ View report in BundleMon website ➡️ |
We need to update cozy-scripts to 8.4.0 to support the lib.
We can just replace the previous search view by the new one easily because both are full page modal.
stop blocking PR for +4kB, use bundlemon monitor to check weight peaks
To do