Skip to content

Conversation

@Kavantix
Copy link

As this repository hosts two packages, please ensure the PR title starts with the name of the package that it relates to using square brackets (e.g. [flutter_local_notifications]). The contribution guidelines can be found at https://github.com/MaikuB/flutter_local_notifications/blob/master/CONTRIBUTING.md. Please review this as it contains details on what to follow when submitting a PR.

@Kavantix Kavantix requested a review from JeroenvdV April 15, 2022 12:27
AlarmManager alarmManager = getAlarmManager(context);
for (String id : jsonNotifications.keySet()) {
// TODO filter out non-matching IDs
int intId = Integer.parseInt(id.substring(SCHEDULED_NOTIFICATIONS_ID.length() - 1));

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should not be - 1

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.

3 participants