delphiki / hass-pronote

Pronote integration for Home Assistant
58 stars 9 forks source link

Ajout de la séléction du trimestre ? [évaluations, absences] #61

Open solarpush opened 9 months ago

solarpush commented 9 months ago

Bonjour, d'abord merci pour votre travail c'est top,

en regardant un peu je penssais à une erreur en voyant que les evalutions était indisponible dans la liste des sensor (présent mais indispo)

En regardant le code je pense à présent qu'il s'agit simplement d'une entré vide car nous sommes pile dans la transition du second trimestre.

J'imagine que cette fonction ne retourne que la periode courante :

def get_evaluations(client):
    evaluations = client.current_period.evaluations
    evaluations = sorted(evaluations, key=lambda evaluation: (evaluation.name))
    return sorted(evaluations, key=lambda evaluation: (evaluation.date), reverse=True)

est- il possible d'imaginer récupérer toutes les périodes non vides et les attribuer à des sous-sensor ? ou garder la structure sensor.pronote.xxx_evaluations qui reste pour le current et ajouter simplement :

et modifier


def get_evaluations(client, period):
    evaluations = client.[period].evaluations
    evaluations = sorted(evaluations, key=lambda evaluation: (evaluation.name))
    return sorted(evaluations, key=lambda evaluation: (evaluation.date), reverse=True)

et


try:
            self.data['evaluations'] = await self.hass.async_add_executor_job(get_evaluations, client, "current_period")
        except Exception as ex:
            _LOGGER.info("Error getting evaluations from pronote: %s", ex)

try:
            self.data['evaluations_t1'] = await self.hass.async_add_executor_job(get_evaluations, client, "trimestre1")
        except Exception as ex:
            _LOGGER.info("Error getting evaluations from pronote: %s", ex)

//etc

à noter que je ne connais pas du tout python c'est pourquoi je ne me lance pas dans une PR, je n'ai pas du tout épluché le requetes pronote et je ne comprend pas bien cette fonction non-plus : self.hass.async_add_executor_job(get_evaluations, client, "trimestre1")

Merci en tout cas cet ajout serais un plus mais c'est déjà très bien.

plouf34 commented 6 months ago

Meme problème que le précédent; le trimestre n'est pas fini, mais il n'affiche plus les notes du trimestre 2

MickaelDENIS commented 6 months ago

Il serait intéressant de pouvoir avoir tous les trimestres consultable et pas seulement le dernier. à voir si les adaptations qui sont proposés sont possible