laravel-notification-channels / onesignal

OneSignal notifications channel for Laravel
MIT License
283 stars 119 forks source link

Not Support with Laravel 6.0.3 #101

Closed ahmedkamel1192 closed 5 years ago

ahmedkamel1192 commented 5 years ago

i have got this error after run composer command "composer require laravel-notification-channels/onesignal"

Your requirements could not be resolved to an installable set of packages.

Problem 1

LKaemmerling commented 5 years ago

As you can see in #98, we're working on it

LKaemmerling commented 5 years ago

v2.0.0 was just released which support Laravel 6.0

ahmedkamel1192 commented 5 years ago

v2.0.0 was just released which support Laravel 6.0

is it take time to be shown in packagist ?

LKaemmerling commented 5 years ago

Really strange, i don't know why the sync does not work. We doesn't have control over this packagist account. @atymic can you look into this?

atymic commented 5 years ago

A bunch of other repos have the issue, we need @themsaid to fix it (he has the packagist account) :(

atymic commented 5 years ago

I'm just re-opening this while we wait for packagist to be fixed.

r94ever commented 5 years ago

Anyone know what happened with packagist?

Lloople commented 5 years ago

They changed the way external apps connect with GitHub API, so you have to give access again. But only the packagist account owner can do that so we need @themsaid in order to do that 😅

r94ever commented 5 years ago

They changed the way external apps connect with GitHub API, so you have to give access again. But only the packagist account owner can do that so we need @themsaid in order to do that

And @themsaid hasn't done it yet, has him?

atymic commented 5 years ago

No, there a bunch of other repos with the issue as well https://github.com/laravel-notification-channels/laravel-notification-channels.github.io/issues/25

AshPowell commented 5 years ago

No, there a bunch of other repos with the issue as well laravel-notification-channels/laravel-notification-channels.github.io#25

Is it also worth tagging Taylor in the todo you created to get this moved along? I'm guessing it's stopping a fair few apps from being updated that rely on these push notification channels :/

atymic commented 5 years ago

AFAIK taylor isn't involved in the notification channels (apart from working with @themsaid)

AshPowell commented 5 years ago

AFAIK taylor isn't involved in the notification channels (apart from working with @themsaid)

I just thought maybe if we could get Taylor to drop him a PM that's all - I imagine there are a bunch waiting on these now - can't update to v6 because of this 😕

atymic commented 5 years ago

Github hooks should be syncing with packagist properly now. If there are any issues in the future ping me 😄