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

Chapter 03 - Fix 'label: false' #19

Closed

Conversation

abinoam
Copy link
Contributor

@abinoam abinoam commented Jun 9, 2016

Hi @apotonick,

I have updated the Simple Form Foundation templates and trimmed the repetitive 'label: false' from some views.
I have also used locale files to define placeholders.

After I have began to work on this I could notice that @smathy have already worked on it on PR #9. But unfortunately his branch is stale and with conflicts.

I have also walked through each chapter from 3 to 11 merging the changes from the previous chapter and fixing anything new that appears on each chapter.

I have fixed the "no labels" issue on chapter-10 as your comment at #9 (comment)

I'm opening this PR for chapter-03 branch. But I don't know if you want me to open a PR for each chapter branch or you're gonna just fetch my repo and merge locally.

Each branch comes from its corresponding chapter branch and can be merged without conflicts.

All branches ends with feat-fix-form
See: https://github.com/abinoam/gemgem-trbrb/branches/active

@apotonick
Copy link
Owner

Hey, thanks @abinoam ❤️ I decided that I don't want to promote SimpleForm anymore and in the next book we will use Formular.

@apotonick apotonick closed this Jun 15, 2016
@abinoam
Copy link
Contributor Author

abinoam commented Jun 15, 2016

No problem! 👍

Still slowly re-reading Trailblazer book. More to come!

Em Qua, 15 de jun de 2016 06:26, Nick Sutterer [email protected]
escreveu:

Closed #19 #19.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#19 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAhrbpx8gtlCxtsxZjb4H1jgLGMoZgH-ks5qL8VYgaJpZM4IyNZk
.

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