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

Birthday #398

Merged
merged 6 commits into from
Apr 22, 2024
Merged

Birthday #398

merged 6 commits into from
Apr 22, 2024

Conversation

STT65
Copy link

@STT65 STT65 commented Apr 22, 2024

Hallo @dbambus, der Branch "Birthday" ist erneut hochgeladen (mit kleinen Nacharbeiten). Hoffentlich kannst du damit arbeiten!?

New Feature: Animation at birthdays
Implementation phase 1: Web interface
- Added page “Birthdays”
- Two different birthday dates can be entered and stored to EEPROM
- Birthday dates are accessible via G.birthday1 and  G.birthday2
Implementation phase 2: Before rework of Transition::isSilvester
- Made 2 bugfixes for Silvester Animation
- Made Uhrtypes ready for using the words "Happy Birthday"
Implementation phase 3: Almost ready, open topics:
- While FireWorks switch on "Happy Birthday" (without time)
- Change FireWorks interval from 5 to 60 minutes
@dbambus dbambus changed the base branch from main to Birthday April 22, 2024 22:31
@dbambus dbambus merged commit 5a88a45 into ESPWortuhr:Birthday Apr 22, 2024
1 of 2 checks passed
@dbambus
Copy link
Collaborator

dbambus commented Apr 22, 2024

Ich habe das jetzt trotz einiger Lint Fehler in einen Branch von ESPWordclock gemergt. Vielen Dank erst mal für die Hilfe mit dem Code.

In den nächsten Tagen werde ich mir den Code genauer anschauen und bei mir testen, bevor er in den Main wandert. Da ich noch prüfen muss ob alles so funktioniert wie ich mir das gedacht habe und ob meine Konfliktlösung so funktioniert.

Ich werde dich hier auf dem Laufenden halten.

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

Successfully merging this pull request may close these issues.

None yet

2 participants