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.

Sign up for GitHub

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

Jump to bottom

Fix for RTC not allowing matching more than one field on Alarm #156

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Fix for RTC not allowing matching more than one field on Alarm #156

merged 1 commit into from
Oct 2, 2023

Conversation

maidnl
Copy link
Contributor

@maidnl maidnl commented Oct 2, 2023

This fixes issue on RTC: the set Alarm only worked for 1 field (seconds or minutes or hours and so on...).

#154

@maidnl maidnl requested a review from facchinm October 2, 2023 10:18
@facchinm facchinm merged commit b7912da into arduino:main Oct 2, 2023
@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: code Related to content of the project itself labels Oct 2, 2023
@per1234 per1234 linked an issue Oct 2, 2023 that may be closed by this pull request
RTC.setAlarmCallback does not allows matching on more than one field #154
Closed
@maidnl maidnl deleted the fix_rtc_issue_cb_not_allowing_more_fields branch October 10, 2023 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@facchinm facchinm Awaiting requested review from facchinm

Assignees
No one assigned
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

RTC.setAlarmCallback does not allows matching on more than one field
3 participants
@maidnl @per1234 @facchinm