feat: Show timeseries in harvest's data tab #1271
Conversation
ptbrowne
changed the title
This way, each application can choose which data card are displayed in harvest. This is helpful since data cards can require large deps like leaflet.
Apps will import datacards directly, the import path needs to be short and clear
Mock konnectors need an attributes attr since it is used in the DataTab
This helps to have the content not jump when the timeseries have been fetched
| return Object.values(permissions) | ||
| .map(permission => doctypeToDataCard[permission.type]) | ||
| .filter(Boolean) | ||
| } |
There was a problem hiding this comment.
C'est un peu dommage de se baser sur les permissions pour afficher ou non un bloc de données.
On devrait plutôt se baser sur le manifest du connecteur, et le manifest devrait déclarer les blocs qu'il veut afficher dans Harvest, un peu à la manière des champs de login. Parce que c'est pas parce que mon connecteur a une permissions sur les fichiers que j'ai envie qu'il affiche le bloc de fichiers, ça devrait être décorrélé.
A garder en tête pour les futures itérations sur cette partie, j'arrive après la bataille ici.
There was a problem hiding this comment.
On devrait plutôt se baser sur le manifest du connecteur, et le manifest devrait déclarer les blocs qu'il veut afficher dans Harvest,
Le problème dans ce cas là, c'est repasser sur tout les connecteurs déjà existant et mettre cette déclaration. Personnellement je pense que ça suffirait de ne prendre que la première permission qui match et optionnellement de pouvoir dire sur une permission que tu ne veux pas qu'elle soit affichée en tant que datacard. Ou de pouvoir spécifier en dur le visualiseur que tu veux pour tes données.
datacards: {
'io.cozy.permissions': false,
'io.cozy.files': 'FileDataCard2' // already existing in harvest
}
Parce que c'est pas parce que mon connecteur a une permissions sur les fichiers que j'ai envie qu'il affiche le bloc de fichiers, ça devrait être décorrélé.
C'est à débattre, je vois pas très bien un usecase ou en tant que développeur de connecteur tu veux afficher un viewer différent.
There was a problem hiding this comment.
J'ai fait une issue : #1275
There was a problem hiding this comment.
On en discutera en atelier, car y a déjà pas mal de travail fait sur ce sujet, ce sera l'occasion de fournir toutes les billes à tout le monde
There was a problem hiding this comment.
C'est à débattre, je vois pas très bien un usecase ou en tant que développeur de connecteur tu veux afficher un viewer différent.
Pour info, sur un connecteur qui aurait une permission sur un docType mais sans pour autant une volonté d'afficher le viewer sur cette donnée, il suffit de prendre le cas des connecteurs de factures qui demandent la permission sur les données bancaires pour créer leur bills.
Je ne veux pas avoir d'encart 'Mes données bancaires' même si le connecteur demande les permissions sur ce docType.
Add a way to display data cards in the data tab of harvest.
Here, only a geo data card is implemented for the io.cozy.timeseries.geojson. An app wanting to use this will have to install leaflet and material-ui/lab.
Related: cozy/cozy-home#1566