-
Notifications
You must be signed in to change notification settings - Fork 127
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
Support better support collection of tokens #130
Conversation
ping @atymic |
I can merge this myself. However I am unable to test this as I don't currently have access to an Android project. As noted in the other issue I'd love some people to test this and see if it works well. It drops some older PHP versions and introduces other changes. |
@dwightwatson Anything in particular you would want to test? I tested basic sending on Android with setting a title and body and it was sent fine 👍 |
Nope - that's a great sanity check - thanks for that. Maybe I'll merge this and tag as v3, that way it can live alongside v2 and both will support Laravel 10. Can iron out any issues as they arise but users can always fall back to v2 if they need. |
spatie/enum
dependencyVisibility
enum