codecurious-bln / basic-curriculum Goto Github PK

Add screenshots of the resulting views so that an attendee can compare their version

Right now we always included bootstrap straight from railsgirls, they changed it to bootstrap 3 so if we keep it we might need to adapt. I'd be pro on using it from local, but that might be some extra headaches.

Either way, using the new version 3 would be nice!

In the Array section:

... So the last one would be planets.[3]

Should be

... So the last one would be planets[3]

without the dot

Jan did quite some work on his fork. As the tutorial is currently down from the website I think it'd be nice to get it back up so working on getting his changes (conversion to markdown etc.) merged in here. Or maybe make it a new repo since it's more of a replacement than a merge.

Not much time now (need it for my course though so will do some work on it) - but after my thesis and before eurucamp I'll hopefully be able to help with this. :)

Cheers,
Tobi

In some code examples there are plus signs which cause errors

planets.each do |x| puts + x end

For including the rails girls css file, why not use stylesheet_link_tag as well, rather than literal HTML?
default generated output is HTML5 now, so should be for consistency

I was wondering if it would make sense to configure the route in step 3 and not in step 5. If you do so, you can see a result immediately and don't have to wait until the end of the tutorial.

What do you think?

I just openend an issue in the RailsInstaller repository and asked when they plan to support Rails 4.
railsinstaller/railsinstaller-nix#20

It seems that the new version of gmaps4rails is a complete rewrite. Step 6 will probably need to be rewritten to reflect that, as it is not working right now. We ran into the problem in todays hackday.

Everything after (and including) rails generate gmaps4rails:install isn't working.

In the attendee tutorial url <%= image_tag(@attendee.picture_url, :width => 600) if @attendee.picture.present? %> should be changed to <%= image_tag(@attendee.picture, :width => 600) if @attendee.picture.present? %>

Here is the link to the website:

http://railsgirlsberlin.de/apptutorial/

The HTML that is in the repo seems to not be hand-written, and editing it is not very pleasant.

An easy solution would be to convert everything to Markdown from which we can generate the HTML. But I'd like to also talk here about how these files are published.

I'm guessing from the file names that someone copy pastes them to Wordpress? Is this something we can automate?

Having it automated will ensure that updates quickly make it to the website.

I quickly went through the tutorial

https://github.com/plexus/railsgirls_rails4

The only problem I had was that somehow the map has zero height, so
it's invisible. Setting #map { height: 500px; } solved that.

• removing index.html is no longer necessary, just setting a root
  route is enough (Done)
• for including the rails girls css file, why not use
  stylesheet_link_tag as well, rather than literal HTML?
• default generated output is HTML5 now, so
  should be
  for consistency
  => See #5

Hi all,

so attr_accessible was replaced by strong parameters, which basically is the same thing. Just in the controller and more flexible. Takes some time getting used to.

Some code listings probably need to get changed. I believe now the default scaffold also generates a before_action (formerly known as before_filter) to get the resource for show, edit and destroy.

There could be problems with turbo links, those sometimes break the JavaScript (depending on events etc.) - we would have to check if they break the google maps thingy.

Also gem compatibility but carrierwave should work with rails 4.

Otherwise I think we're good (of the top of my head) but not certain,

The master branch is outdated as far as I can tell. the gh-pages branch seems to be the new hotness. E.g. it'd be good to make it the default branch for everyone to see (and not miss that like I did). E.g. default branch is the branch shown when you visit this repository on github.

Everyone alright with that?

... or at least have students try it, play with it and possibly help to improve it.

• http://teams.railsgirlssummerofcode.org/teams/10
• https://github.com/railsgirls-generator-app/railsgirls-app
• http://railsgirls-generator-app.github.io/railsgirls-app

@pxlpnk knows all the details :)

As Tobi already mentioned the google map does not appear in the browser.
Developer console:

Uncaught ReferenceError: Gmaps is not defined