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

Replaced raw PHP in the views with standard blade syntax #27

Closed
wants to merge 1 commit into from
Closed

Replaced raw PHP in the views with standard blade syntax #27

wants to merge 1 commit into from

Conversation

moebrowne
Copy link
Contributor

No description provided.

@nWidart
Copy link
Member

nWidart commented Nov 29, 2015

Hello,

Thank you for the contribution. But it was an intended move on using native php over blade, a personal preference. 😄

@nWidart nWidart closed this Nov 29, 2015
@joshbrw
Copy link
Member

joshbrw commented Nov 29, 2015

I'm +1 for using Blade. Vanilla PHP is ugly in views and Blade is a Laravel
standard.
On Sun, 29 Nov 2015 at 7:27 a.m., Nicolas Widart notifications@github.com
wrote:

Hello,

Thank you for the contribution. But it was an intended move on using
native php over blade, a personal preference. [image: 😄]


Reply to this email directly or view it on GitHub
#27 (comment).

@nWidart
Copy link
Member

nWidart commented Nov 29, 2015

Vanilla PHP is ugly

Personal preference. You're free to use blade tags on your own modules.

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.

None yet

3 participants