Closed gruoner closed 2 years ago
zreptil
commented
2 years ago Änderungen von diesem Ausmass und ohne Berücksichtigung der vorhandenen Lauffähigkeit werden auf keinen Fall im Nightscout Reporter landen. Für solche Fälle bitte ein eigenes Repository aufmachen und dort den Code selbst verwalten.
gruoner
commented
2 years ago darum hab ich's dir ja auch per PR geschickt. Dann kannst du das Delta in deinen Code übernehmen und es funktioniert sofort.
Aktuell liegt es in einem eigenen Repo (nämlich meinem) aber dieser Zustand ist auf Dauer unhaltbar und erzeugt von Release zu Release immer mehr Aufwand (meist Unsinnigen, wenn immer wieder Zeilenumbrüche entfernt und wieder eingefügt werden).
Die nächsten zwei grossen Epics sind gerade am Entstehen (Multi-Insulin hängt noch an den Nightscout-Changes zur Insulin-API und Multi-Carbs hängt vor allem an den GUI's im xDrip, die gefallen mir noch nicht). Sobald die "fertig" und den Masterbranches sind, fliesst dieses ganze "Zeugs" auch in den NightscoutReporter und dann wird die Anpassung wirklich gross - da sind die Clarity-Changes "Kindergarten".
Dafür aber nun eine komplette Konkurrenz zum NSR aufzumachen, ist genauso unsinnig, weil irgendwann muss sich dieses Anpassen und Weiterentwickeln auch mal einem konsolidierten Ende entgegen bewegen damit die Leute es auch in nennenswertem Masse benutzen können (wir machen das Ganze ja nicht nur für uns, sondern weil wir an
Am 22.08.22 um 13:38 schrieb Andi:
Änderungen von diesem Ausmass und ohne Berücksichtigung der vorhandenen Lauffähigkeit werden auf keinen Fall im Nightscout Reporter landen. Für solche Fälle bitte ein eigenes Repository aufmachen und dort den Code selbst verwalten.
— Reply to this email directly, view it on GitHub https://github.com/zreptil/nightscout-reporter/pull/63#issuecomment-1222233254, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGW7A6XGLKRM373W6T57DDV2NRB3ANCNFSM5ZEJ5BSA. You are receiving this because you authored the thread.Message ID: @.***>
Wie im #55 angekündigt hier der PR für das Feature - es läuft auf dem aktuellen Masterbranch, aber sobald der sich weiterentwickelt, kann es sein, dass es nicht mehr zusammenpasst.....