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

Make use of helpers function and remember to make add_debug true on production #153

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Masei25
Copy link

@Masei25 Masei25 commented Oct 20, 2022

Some more contribution.
#Just a contribution to the document, ensuring the app_debug is always set to true after deployment.
#Using the helper function available on the Illuminate/Support instead of making a new custom helper function

@spekulatius
Copy link
Contributor

I wouldn't consider setting the debug mode to false on prod as a best practice but more as a general requirement. Best practices cover parts where there are options to choose from while leaving debug on true would be simply wrong.

On the string function, I agree it's something to consider. But it's already covered in the DRY section for my feeling. What do you think?

@Masei25
Copy link
Author

Masei25 commented Oct 23, 2022

I feel the DRY section talks more about reusing code you've written within the application and not really about making use of the built-in functions available in Laravel.
As Laravel has numerous methods available globally within the applications that can be called anywhere within the application

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