mawaqit / home-assistant

Mawaqit integration - salat time and nearest mosque - in Home Assistant

60 stars 16 forks source link

Issue #36 - Added Jumu'a sensors #40

Closed moha-tah closed 1 year ago

moha-tah commented 1 year ago

Issue #36 :

C'est ma première pull request, j'aimerais bien avoir ton avis sur ce code pour le commit sur le master car je suis pas très confiant et c'est possible que j'ai oublié quelque chose.

Exemple dans l'affichage UI :

Affichage dans les settings :

ibrahim-zehhaf-mawaqit commented 1 year ago

Très bon début @mohaThr félicitation Alors je ne suis pas le mieux placer pour review ta pr je ne suis pas un spécialiste de python mais j'essaie 2 petites remarques

Merci d'utiliser l'anglais pour les échanges afin de faciliter la discussion avec des anglophones
Penses à lier a PR à l'issue pour un meilleur suivi et le fonctionnement de l’automatisation github

ibrahim-zehhaf-mawaqit commented 1 year ago

It's ok now, installed and it works, I will test it for a few days before publishing insh'Allah