laravel-fcm-notification icon indicating copy to clipboard operation
laravel-fcm-notification copied to clipboard

Laravel 10.x Compatibility

Open laravel-shift opened this issue 2 years ago • 5 comments

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.

Before merging, you need to:

  • Checkout the l10-compatibility branch
  • Review all comments for additional changes
  • Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

laravel-shift avatar Jan 31 '23 13:01 laravel-shift

:warning: The latest version of Laravel has a dependency for phpunit/phpunit of ^9.5.10. If you would like to also upgrade your tests, you may run the PHPUnit 9 Shift for free.

laravel-shift avatar Jan 31 '23 13:01 laravel-shift

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.

To do so, temporarily add Shift's fork to the repositories property of your composer.json:

{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-fcm-notification.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "benwilkins/laravel-fcm-notification": "dev-l10-compatibility",
    }
}

Finally, run: composer update

laravel-shift avatar Jan 31 '23 13:01 laravel-shift

Hi guys, thank you for this package! any chance this(support for laravel 10) is going to be resolved soon?

njenga55 avatar Feb 17 '23 14:02 njenga55

Hi! As poster above, thanks for this package. Any chance that this pull request can be merged so we can use it with Laravel 10?

Thanks!

vilhelmjosander avatar Feb 27 '23 19:02 vilhelmjosander

Hey @benwilkins, could this get merged? Would be really nice! For the other people here: If this project is abandoned, a switch to https://github.com/ankurk91/fcm-notification-channel might be a good idea - especially because it uses the newer v1 API for Firebase.

mwllgr avatar May 07 '23 10:05 mwllgr