-
Notifications
You must be signed in to change notification settings - Fork 13
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
Guide: Design First Laravel #167
Guide: Design First Laravel #167
Conversation
✅ Deploy Preview for bump-content-hub ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Phil! From my perspective, there's just a few minor suggestions 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please just rebase so your last merged article and any other change are reflected on the PR?
Co-authored-by: Christophe Dujarric <[email protected]>
50a439f
to
ea0a8f0
Compare
* rough draft ready for review * Apply suggestions from code review Co-authored-by: Christophe Dujarric <[email protected]> * update github links to bump-sh-examples --------- Co-authored-by: Christophe Dujarric <[email protected]>
This is a rough draft, as in its functionally complete but the wording is probably not great especially towards the end. If you folks can make pointers and feedback to stuff you like or don't I'll get it fixed up.