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

Conversation

masterix21
Copy link

To create an app from command line:
php artisan websockets:app:create

To manage all existing apps, use the web panel from your browser:
/laravel-websockets/admin

@vesper8
Copy link

vesper8 commented Jan 16, 2019

looks really useful! thanks for working on this!

I have two comments.

First, instanciate is misspelled, it should be instantiate

Second, in all of my websocket apps I use a conditional when setting the host, like this:

'host' => env('APP_ENV') === 'production' ? 'socket.mydomain.com' : 'socketmaster.dev',

Can your addition have any way of handling such a case ?

@masterix21
Copy link
Author

masterix21 commented Jan 16, 2019

@vesper8 thanks for the first comment.

About the second, i think that if you're not using the same database on production and local, the problem is not real.

@vesper8
Copy link

vesper8 commented Jan 16, 2019

@masterix21 good point! didn't think of that

is it possible to enable/disable apps without deleting them?

@masterix21
Copy link
Author

App model uses SoftDelete trait, but i will manage better enable/disable feature.

Thanks

@masterix21
Copy link
Author

  • Added enable/disable apps feature
  • Added pagination to apps index
  • Added search to apps index

@veneliniliev
Copy link

this is great feature... please merge it :)

@eldor
Copy link
Contributor

eldor commented Feb 7, 2019

Until this has been merged or if you want to do it in Nova you can use this package
https://packagist.org/packages/vemcogroup/nova-websockets

@rennokki rennokki added the rebase Rebase is needed label Aug 13, 2020
@rennokki
Copy link
Collaborator

@masterix21 I assume you need to update your fork's master branch with the current one.

@masterix21 masterix21 closed this Apr 7, 2021
@masterix21 masterix21 deleted the feature-database-apps branch April 7, 2021 15:41
@EndOfSource EndOfSource mentioned this pull request May 27, 2021
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
rebase Rebase is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.