Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Events werden nicht korrekt ausgewertet #674

Open
cvoelkel76 opened this issue Jun 12, 2024 · 2 comments
Open

Events werden nicht korrekt ausgewertet #674

cvoelkel76 opened this issue Jun 12, 2024 · 2 comments

Comments

@cvoelkel76
Copy link
Contributor

Wir nutzen unseren Kalender zur Heizungssteuerung.

Kalenderevents:
Urlaub Frau --> Büro Frau wird nicht geheizt
Urlaub Mann --> Büro Mann wird nicht geheizt
Urlaub --> beide Büros werden nicht geheizt

Problem:
Der Kalendereintrag "Urlaub Mann" stellt die Events "Urlaub Mann" und "Urlaub" auf true. Somit bleibt das Büro meiner Frau kalt, obwohl nur ich Urlaub habe.

Also Workaround könnte man zwar statt "Urlaub" zum Beispiel "Urlaub beide" eintragen, aber so wirklich schön ist das nicht.

Ich habe den Adapter für mich angepasst, hält natürlich nur bis zum nächsten update:

692 async function checkForEvents(reason, event, realnow) {

701     // check if event exists in table
702     for (let i = 0; i < events.length; i++) {
703         const ev = events[i];

704         //if ((reason.includes(ev.name)) || (ignoreCaseInEventname && (reason.toLowerCase().includes(ev.name.toLowerCase())))) {

705         if ((reason === ev.name) || (ignoreCaseInEventname && (reason.toLowerCase() === ev.name.toLowerCase()))) {

706             // check if event should shown
707             result = ev.display; 
708             adapter.log.debug(`found event in table: ${ev.name}, day=${ev.day}`);

Eventuell könnte man das generell ändern wenn gewünscht.

Grüße
Christian

@klein0r
Copy link
Contributor

klein0r commented Jun 12, 2024

Eventuell könnte man das generell ändern wenn gewünscht.

Wenn, dann nur konfigurierbar. Sonst würde sich ja das Verhalten bei allen ändern. Ich würde empfehlen, die Events "Urlaub" anders zu nennen. Also z.B. "Urlaub generell" oder "Urlaub alle".

@Baenker
Copy link

Baenker commented Nov 1, 2024

Ich löse es wie von kleinOr beschrieben. Es gibt bei mir Urlaub Mann, Urlaub Frau und Urlaub Beide. Wenn nur Urlaub dort steht würde ich mich fragen ob ich oder meine Frau vergessen hat den Namen anzufügen oder ob wirklich Beide Urlaub haben. Man könnte halt auch generell auf Urlaub Beide verzichten wenn jeder seinen Urlaub einträgt. Ist bei Homeoffice ja auch nicht anders.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants