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

build(deps): bump flutter_local_notifications from 17.2.4 to 18.0.1 in /packages/altfire_messenger #108

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 11, 2024

Bumps flutter_local_notifications from 17.2.4 to 18.0.1.

Release notes

Sourced from flutter_local_notifications's releases.

flutter_local_notifications-v18.0.1

  • Added upper bound constraints for timezone dependency. This is to allow the plugin and apps pull version 0.10.x if constraints are satisified. Future releases may bump the minimum timezone dependency but this approach is being used at the moment as version 0.10.0 of the time timezone package introduces a dependency on http package. Directly bumping the flutter_local_notifications dependency on timezone could have introduced dependency conflicts

flutter_local_notifications-v18.0.0

  • Breaking changes Bumped minimum Flutter SDK requirement to 3.13. Consequently the minimum OS requirements for each platform has been updated as well
    • [Android] minimum Android version is now 4.4 (API level 19)
    • [iOS] minimum iOS version is now 11
    • [macOS] minimum macOS version is now 10.14
  • [Android] Breaking change removed the deprecated androidAllowWhileIdle parameter from zonedSchedule() and periodicallyShow() methods. androidScheduleMode is now a required parameter
  • Breaking change plugin has been fixed with regards to how it registers the platform-specific implementations of the plugin. Thanks to the PR from Kate. Any written tests done on the [FlutterLocalNotificationsPlugin] will need to manually call the registerWith() method that has been added to each implementation. This affects the following platforms/classes where the registerWith() method was added
    • [Android] AndroidFlutterLocalNotificationsPlugin
    • [iOS] IOSFlutterLocalNotificationsPlugin
    • [macOS] MacOSFlutterLocalNotificationsPlugin
  • [iOS] Breaking change Removed onDidReceiveLocalNotification callback as this was only relevant on iOS versions older than 10
  • Fixed example app to have the appropriate permissions for foreground services
  • Updated readme when it comes to setting up the AndroidManifest.xml file to include details of what's needed for foreground services. Note these details were already available in the API docs

flutter_local_notifications-v18.0.0-dev.3

  • Included changes from 17.2.4 release that was done to add in a privacy manifest file to macOS implementation

flutter_local_notifications-v18.0.0-dev.2

  • [iOS] Breaking change Removed onDidReceiveLocalNotification callback as this was only relevant on iOS versions older than 10

flutter_local_notifications-v18.0.0-dev.1

  • Breaking changes Bumped minimum Flutter SDK requirement to 3.13. Consequently the minimum OS requirements for each platform has been updated as well
    • [Android] minimum Android version is now 4.4 (API level 19)
    • [iOS] minimum iOS version is now 11
    • [macOS] minimum macOS version is now 10.14
  • [Android] Breaking change removed the deprecated androidAllowWhileIdle parameter from zonedSchedule() and periodicallyShow() methods. androidScheduleMode is now a required parameter
  • Breaking change plugin has been fixed with regards to how it registers the platform-specific implementations of the plugin. Thanks to the PR from Kate. Any written tests done on the [FlutterLocalNotificationsPlugin] will need to manually call the registerWith() method that has been added to each implementation. This affects the following platforms/classes where the registerWith() method was added
    • [Android] AndroidFlutterLocalNotificationsPlugin
    • [iOS] IOSFlutterLocalNotificationsPlugin
    • [macOS] MacOSFlutterLocalNotificationsPlugin
  • Fixed example app to have the appropriate permissions for foreground services
  • Updated readme when it comes to setting up the AndroidManifest.xml file to include details of what's needed for foreground services. Note these details were already available in the API docs
Commits
  • c15783f [flutter_local_notifications] fixed 18.0.1 changelog (#2452)
  • d1ee43d [flutter_local_notifications] prepare for 18.0.1 release (#2451)
  • e5e9b51 Bump timezone from 0.9.4 to 0.10.0 in /flutter_local_notifications (#2445)
  • 94a1170 [various] prepare for 18.0.0 release (#2442)
  • 15fe930 prepare for 18.0.0-dev.3 release (#2437)
  • 1da4bb7 added privacy manifest file to macOS implementation (#2436)
  • 8b8669e [flutter_local_notifications] removed all code related to UILocalNotification...
  • a3d3323 split out format workflow to occur on push (#2433)
  • 7265ae2 bumped dependency to linux 5.0.0 prerelease (#2422)
  • cc67070 updated various plugins for prerelease (#2421)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested a review from a team as a code owner November 11, 2024 13:08
@dependabot dependabot bot added dart Pull requests that update Dart code dependencies Pull requests that update a dependency file labels Nov 11, 2024
@dependabot dependabot bot requested review from naipaka and removed request for a team November 11, 2024 13:08
@dependabot dependabot bot requested a review from boywithdv November 11, 2024 13:08
@boywithdv
Copy link
Contributor

@naipaka

As it stands, we are done with setting public_member_api_docs to false and changing the deprecated reportErrorForNode to atNode.
The only other thing to do is to remove or modify the onDidReceiveLocalNotification property of flutter_local_notifications.
The version of flutter_local_notifications is now 18 and onDidReceiveLocalNotification has been removed.
Although onDidReceiveLocalNotification is a property that applies to versions prior to iOS 10, we could not find a property that could be substituted for onDidReceiveLocalNotification when it is removed.
If there is another property that could be used as a substitute, it should be used, or if not, onDidReceiveLocalNotification should be deleted and merged.

I have not found a good way to handle this on my end, and would appreciate it if you have a better way to handle this.

@naipaka
Copy link
Contributor

naipaka commented Nov 21, 2024

I'll handle it on my end! 👍

@naipaka naipaka requested review from naipaka and boywithdv and removed request for naipaka and boywithdv November 21, 2024 09:13
@naipaka naipaka force-pushed the dependabot/pub/packages/altfire_messenger/flutter_local_notifications-18.0.1 branch from 4df5486 to 9d78d88 Compare December 2, 2024 08:28
Bumps [flutter_local_notifications](https://github.com/MaikuB/flutter_local_notifications) from 17.2.4 to 18.0.1.
- [Release notes](https://github.com/MaikuB/flutter_local_notifications/releases)
- [Commits](MaikuB/flutter_local_notifications@flutter_local_notifications-v17.2.4...flutter_local_notifications-v18.0.1)

---
updated-dependencies:
- dependency-name: flutter_local_notifications
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@naipaka naipaka force-pushed the dependabot/pub/packages/altfire_messenger/flutter_local_notifications-18.0.1 branch from 9d78d88 to a42f3ca Compare December 2, 2024 08:30
@naipaka naipaka enabled auto-merge (squash) December 2, 2024 08:33
@naipaka naipaka merged commit 8bfff27 into main Dec 2, 2024
4 checks passed
@naipaka naipaka deleted the dependabot/pub/packages/altfire_messenger/flutter_local_notifications-18.0.1 branch December 2, 2024 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dart Pull requests that update Dart code dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants