Skip to content

Conversation

@danalamo
Copy link

@danalamo danalamo commented Apr 21, 2025

My Fork with Updates

@gogl92
Copy link

gogl92 commented Jun 12, 2025

I'm having issues with Laravel 12 (lib conflicts while upgrading Laravel), can you help us to review this PR? @clemblanco @oaklees @HasnatH

thank you!

@clemblanco
Copy link
Contributor

clemblanco commented Jun 12, 2025

Thank you @danalamo

I'm sorry @gogl92 but I don't work at Pod Point anymore therefore I don't have the permission to review you PR but I'm sure my good friends @oaklees or @HasnatH will be able to help.

@danalamo
Copy link
Author

danalamo commented Jun 21, 2025

@gogl92, if you'd like, you can use our fork in the mean time (until they merge this PR) by updating your composer.json like so (this is what our company is doing in our project recently upgraded to Laravel 12):

{
+    "repositories": [
+        {
+            "type": "git",
+            "url": "https://github.com/cleanscript/laravel-aws-pubsub"
+        }
+    ],
    "require": {
        "laravel/framework": "^12.0",
-       "pod-point/laravel-aws-pubsub": "^0.6.0",
+       "pod-point/laravel-aws-pubsub": "dev-main",
        "twilio/sdk": "^6.44.4"
    },
}

You can watch this issue (as we're doing) and change back to the new version on the original repo once this PR is merged.

@renaldyaway
Copy link

@oaklees and @HasnatH
When can you review this PR?

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants