Skip to content
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

Need to install:broadcasting separately? #101

Closed
stephenmeehanuk opened this issue Jun 19, 2024 · 4 comments · Fixed by statamic/cms#10335
Closed

Need to install:broadcasting separately? #101

stephenmeehanuk opened this issue Jun 19, 2024 · 4 comments · Fixed by statamic/cms#10335
Assignees

Comments

@stephenmeehanuk
Copy link

Hi

I installed php please install:collaboration, set everything up

  • I've added the Pusher credentials to my .env
  • Enabled the "Client Events" setting in Pusher Dashboard
  • BROADCAST_CONNECTION=pusher

I've ran php artisan config:cache , but I still see a Attempting websocket connection... message

  • Running Statamic 5.8.0
  • Collaboration 1.0.0

I see this error message in Pusher Dashboard

Cannot broadcast client event (connection not subscribed to channel presence-entry.xxxxxxx-xxx-xxxx
x-0a0b38903058.default)

Dev Tools in the /cp shows a missing file?

broadcasting/auth 404 (Not Found)

I asked about this in Discord
https://discord.com/channels/489818810157891584/1252923239307415653

I was advised to install broadcasting manually.

php artisan install:broadcasting

I did, and that fixed it.

However the docs indicate this is done as part of the php please install:collaboration command.

Is there something wrong with the addon, or do the docs need updating?

@duncanmcclean
Copy link
Member

Statamic's install:collaboration should be running that command behind the scenes, without you needing to do anything (see here for reference).

Which version of Laravel are you using?

@stephenmeehanuk
Copy link
Author

I thought so too.

Laravel Version: 11.10.0
PHP Version: 8.3.8

@stephenmeehanuk
Copy link
Author

Ah, when I ran install:broadcasting I said yes to the 'install node dependencies'. Looks like that's not required.

Line 77

'install:broadcasting',
                    '--without-reverb',
                    '--without-node',

@duncanmcclean
Copy link
Member

Okay, there must be a bug in the command, I'll take a look. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants