delphiki / hass-pronote

Pronote integration for Home Assistant
65 stars 11 forks source link

Add Information And Surveys sensor #27

Closed GaPhi closed 1 year ago

GaPhi commented 1 year ago

Il faut mettre un début à une date négative pour obtenir un résultat

20

vingerha commented 1 year ago

Donc, tant des choses ont changé, j'ai retiré mon PR pour Info/Survey, tu refait la tienne basé sur 080? Autre: il ne faut pas faire trops des choses dans une PR, ça devients très difficile a tester et p.e. dans ce cas...impossible a migrer

GaPhi commented 1 year ago

Merged and adjust to follow last 0.8.0 source refactoring form

GaPhi commented 1 year ago

Donc, tant des choses ont changé, j'ai retiré mon PR pour Info/Survey, tu refait la tienne basé sur 080? Autre: il ne faut pas faire trops des choses dans une PR, ça devients très difficile a tester et p.e. dans ce cas...impossible a migrer

Tu avais réussi à récupérer des données sans mettre de datetime ? De mon côté cela ne retournait rien, d'où la refonte. Il faudrait peut-être retester sans paramètre si cela fonctionne?

vingerha commented 1 year ago

Et oui...


def get_informations(client):
    try:
        informations = client.information_and_surveys()
    except Exception as err:
        _LOGGER.debug(err)
        informations = []
    return sorted(informations, key=lambda information: (information.creation_date), reverse=True)    
GaPhi commented 1 year ago

Je ne comprends pas, cela ne fonctionnait pas mais fonctionne à présent sans limite de date...

Je retire donc la limite des 15 derniers jours, à charge pour les utilisateurs des données de filtrer ce qu'ils souhaitent afficher.

vingerha commented 1 year ago

STP enlever tout ID car ils ne sont pas stable et changent avec chaque connection, ce qui fait une maj des sensors et les automations deviennent impossible

GaPhi commented 1 year ago

Ok, je supprime les "id" s'il ne sont pas stables

vingerha commented 1 year ago

Est-ce que tu peut ajouter 'files' dans homework?

                'files': format_attachment_list(homework.files),

Pour que j'arrive à ça image

GaPhi commented 1 year ago

Devrait être prêt pour une récupération sur main

vingerha commented 1 year ago

Top, merci et maintenant on attend pour quand @delphiki a du temps :)

delphiki commented 1 year ago

Hello @GaPhi, merci pour ta PR, ça m'a l'air good :)