Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Conversation

@dennis1804
Copy link

add support for images

@mathieu21
Copy link

tested and work!

@williamdes
Copy link

Hi !
Feel free to re-open this pull-request to code-lts/Laravel-FCM

This is a fork that will be maintained as long as possible by @code-lts members 💪

🚀 1.4.0 was released See how to install it (composer require code-lts/laravel-fcm)

@williamdes
Copy link

Merged as code-lts#2
Released as v1.5.0

@quocthi2005
Copy link

quocthi2005 commented Mar 4, 2021

Hi !
You can merge it @brozot

@williamdes
Copy link

Hi !
You can merge it @brozot

You should use the code-lts version ^^

@quocthi2005
Copy link

Can I install both? @williamdes

@williamdes
Copy link

Can I install both? @williamdes

No (and yes but do not do that), it makes no sense :)
The code-lts version is a better version of the actual brozot one.

composer require code-lts/laravel-fcm

@quocthi2005
Copy link

But I am using brozot/Laravel-FCM. So when I remove it and use code-lts version will there be an error in brozot version? @williamdes

@williamdes
Copy link

But I am using brozot/Laravel-FCM. So when I remove it and use code-lts version will there be an error in brozot version? @williamdes

I do not see what error you could get, you can just switch to the code-lts version
Here is the diff for the first version I released, as you can see there is no source file changed: 1.3.0...code-lts:1.4.0
So there are compatible :)

@quocthi2005
Copy link

quocthi2005 commented Mar 4, 2021

But I am using brozot/Laravel-FCM. So when I remove it and use code-lts version will there be an error in brozot version? @williamdes

I do not see what error you could get, you can just switch to the code-lts version
Here is the diff for the first version I released, as you can see there is no source file changed: 1.3.0...code-lts:1.4.0
So there are compatible :)

Thanks @williamdes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants