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

[6.x] Fix config not publishing for Laravel Breeze #368

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

Supernova3339
Copy link
Contributor

@Supernova3339 Supernova3339 commented Aug 5, 2024

Removed an additional .php preventing config files from being published.

Summary

This pull request fixes an issue preventing the socialstream.php configuration file from being published

Explanation

There was a typo mistake made in src/SocialstreamServiceProvider.php that prevented the config/socialstream.php file from being published on any installation of Laravel Breeze

  • changed line 191 on src/SocialstreamServiceProvider.php that fixes a typo in the Socialstream code base.

Checklist

  • I've read this template
  • I've checked reviewed this PR myself, ensuring consistency and quality with the rest of the project
  • I've given a good reason as to why this PR exposes new / removes existing user data

Removed an additional .php preventing config files from being published.
@joelbutcher joelbutcher changed the title 🐛 Fix Socialstream configuration file ( socialstream.php ) from not publishing whenever installing Laravel Breeze [6.x] 🐛 Fix Socialstream configuration file ( socialstream.php ) from not publishing whenever installing Laravel Breeze Aug 6, 2024
@joelbutcher
Copy link
Owner

Thanks @Supernova3339

@joelbutcher joelbutcher changed the title [6.x] 🐛 Fix Socialstream configuration file ( socialstream.php ) from not publishing whenever installing Laravel Breeze [6.x] Fix config not publishing for Laravel Breeze Aug 6, 2024
@joelbutcher joelbutcher merged commit d115bc8 into joelbutcher:6.x Aug 6, 2024
38 checks passed
@Supernova3339
Copy link
Contributor Author

Thanks @Supernova3339

No problem! Sorry for the late response, didn't get a notification :/

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 this pull request may close these issues.

2 participants