element that wraps the cart the CSS class carts." The element given the css class “carts” was a
tag. The
was not given a css class, and is not in the preceding code example. You find it back in the _cart.html.erb partial as a basic
tag.
By Developers, For Developers
PDF Pg | Paper Pg | Type | Description | Fixed on | Comments |
---|---|---|---|---|---|
11 | ERROR | Next, run this command to update Rails to the version used by this book: Use: | 2019-06-04 | ||
10 | OK | Open the terminal and run the following command: I had to use “/usr/bin/ruby” instead of “ruby” to get this to work for me on MacOS Mojave. | 2019-10-23 | The official homebrew instructions say ruby. | |
82 | ERROR | You given no link or istructions to download the images (in this case “ridocker.jpg”) in img asset. Conseguence is an action controller exception that no rails experienced people cannot easily resolve. | 2019-10-02 | ||
163 | ERROR | In the file create.js.erb, the cart is stored in a variable: let cart = document.getElementById(‘cart’) The first time an “Add to cart” button is clicked, this works. But subsequent times, an error is thrown because the cart variable is declared again within the same scope. A fix would be to use var instead of let: var cart = document.getElementById(‘cart’) | 2019-10-03 | ||
11 | ERROR | The “brew install chromedriver” in the Installing on Mac OS X outputs this error: $ brew install chromedriver According to this link (removed) that command is Inactive: This link (removed) has a solution: brew cask install chromedriver Note - should all references in the book to Mac OS X now be changed to MacOS? Removed link to Wikipedia article - Tony Dehnke | 2019-06-04 | ||
88 | OK | Change the font in the PDF - the regular body text is very squished and renders poorly (at least for me on MacOS 10.14.4). Can send a screen shot if you like. | 2019-11-15 | Hi, Tony. Please check your settings. It could be the viewing percentage might not be high enough. If you're still having issues after you increase the percentage, please let me know. | |
181 | TYPO | Halfway the second paragraph, a code snippet says: module: @order This should be: model: order (The first argument, module: @order tells the helper…) | 2019-10-17 | ||
10 | ERROR | I start using Ruby and Rails back in 2006. Stopped for a while and I’m now trying to come up to speed with the latest version so I picked the beta version 6. The book says to use the —no-ri and —no-rdoc but with ruby 2.6.3p62 (“gem” version 3.0.3), those options are gone. I looked at the older version of gem (version 2.5.2.3) and they are deprecated in favor of —no-document For newbies, and given that this is in the version 6 of the book, I would put the new rather than the old. | 2019-06-04 | ||
all | OK | More of a question… with the PDF, there are links at the bottom of each page to get me to this form. Can something like that be implemented in the epub format so it shows up in iOS Books? | 2019-10-23 | Hi, Perry. Unfortunately, since the epubs don't have actual pages like the pdfs do, there isn't a way for us to add these inks. | |
11 | ERROR | Repeat of the brew chromedriver issue but now the brew messgae is: brew install chromedriver The two step sequence “worked” in that it installed soemthing :-) | 2019-06-04 | ||
78 | ERROR | Page 78 footnote 3 has a URL to download the images. I get a 403 Forbidden error. The seeds.rb URL (footnote 2) works fine. | 2019-10-02 | ||
99 | OK | Not sure if I have done something wrong, but while running the tests from pages 95-99, I’m getting some fails for the products_controller_test, and I don’t think I should be seeing those at this point?:
F Failure: rails test test/controllers/products_controller_test.rb:18 F Failure: | 2019-11-15 | Sadly, this is not enough information for me to reproduce the problem. | |
110 | TYPO | Should the <main class …. be so far to the right of the rest of the code in the application.html.erb view in from this page? | 2019-10-02 | ||
129 | TYPO | Pretty sure the screen shot is not correct on this page - Should be showing the Add to Cart button, but it’s not. Also - the Add to Cart button still looks the same for me even after adding the CSS formatting on page 128/129 for the button? | 2019-11-15 | This appears to have been fixed in the current beta | |
131 | TYPO | Incorrect Screen Shot - does not show the result of Add to Cart PS - it would be great if we could upload screen shots when submitting these. Also would be nice if we didn’t have to do the Captcha every time, especially if we are logged into our pragprog.com store account when submitting. | 2019-11-15 | This appears to have been fixed in the current beta | |
132 | TYPO | Screen Shot incorrect - does not show the Notice box. | 2019-11-15 | This appears to have been fixed in the current beta | |
11 | SUGGEST | The rbenv install instructions contains the following… | 2019-10-17 | ||
1 | OK | I’m surprised there is no longer a deployment section of the book. And the book “Deploying Rails Applications” hasn’t been updated since its original release in 2008 — over a decade ago. Any suggestions for where to go for current best practices for rails deployment? | 2019-10-23 | These days containers/kubernetes is the way to go, and that topic is way out of scope for a book focused on Rails beginners. | |
1 | 1 | OK | Ditto with the deploy comment. Would love to see some kind of addenda, particularly for the non-devops crowd. | 2019-10-23 | These days containers/kubernetes is the way to go, and that topic is way out of scope for a book focused on Rails beginners. |
292 | TYPO | On page 292 of the book in iBooks, the sentence reads, ‘We’re instead redirecting to the cart, where the cart ID is internal state data residing a cookie.’ It should read, ‘We’re instead redirecting to the cart, where the cart ID is internal state data residing in a cookie.’ The word ‘in’ is missing. | 2019-10-17 | ||
326 | TYPO | On page 326 in iBooks, in the carst.scss file, in the tfoot portion, it states ‘td.price’, where is should state ‘th.price’. This happens twice in this file. | 2019-10-17 | ||
78 | ERROR | Footnote #3 comes back with forbidden status | 2019-10-02 | ||
11 | ERROR | When trying to install ChromeDriver using brew install I get this error message: $brew install chromedriver If, instead I install like this: $brew cask install chromedriver It installs fine. | 2019-06-04 | ||
23 | SUGGEST | The text says: “If you want to enable this server to be accessed by other machines on your network, you can specify 0.0.0.0 as the host to bind to: However, trying to actually reach the Rails app from another computer, I had to add a line to the corresponding configuration (config/environments/development.rb): config.hosts << “#{local_hostname_here}” Actually, the Rails app told me so: Blocked host: #{local_hostname_here} config.hosts << “#{local_hostname_here}” I suggest to add a line or two explaining this to the corresponding section. config.hosts << ‘seaside.local’ | 2019-06-04 | ||
27 | TYPO | “That’s the template file is named hello.html.erb” proabbly should been “That’s why the template file is named hello.html.erb” | 2019-10-02 | ||
11 | ERROR | chromedriver is now a brew cask, | 2019-10-02 | ||
411 | TYPO | “rake” vs “bin/rails” typo. “You can force the database to a specific version by supplying the VERSION= parameter to the rake db:migrate command: depot> bin/rails db:migrate VERSION=20190425000009” Personally, I kinda miss rake, but as you know: “Since Rails 5.0+ has rake commands built into the rails executable, bin/rails is the new default for running commands.” | 2019-10-17 | ||
437 | ERROR | “In Part II, you iteratively built an application, built up test cases along the way, and ultimately deployed it using Capistrano. We designed this application to touch on all aspects of Rails that every developer needs to be aware of.” This is the only location in the entire book where Capistrano is mentioned. | 2019-10-17 | ||
110 | TYPO | javascript_include_tag() should update to javascript_pack_tag() | 2019-10-03 | ||
114 | ERROR | Wrong Screenshot. | 2019-10-17 | ||
163 | TYPO | using ‘let’ will not allow to add more than 1 time. rails6/depot_l/app/views/line_items/create.js.erb | 2019-10-03 | ||
4 | ERROR | rails-dev-box setup process Fails on fresh Ubuntu install. libvirt is not installed in the /var/run/libvirt directory from package manager… vagrant up script breaks with an error. | 2019-11-17 | We have changed the setup to not use rails-dev-box for the next release | |
4 | SUGGEST | Ubuntu 18.04: New vagrant install from aptitude breaks for this command… Must be installed from developer website and it then works. | 2019-11-17 | ||
77 | OK | “bin/rails test” cases error: ➜ depot git:(master) ✗ bin/rails test
Traceback (most recent call last): ’ ’ ’ ’ My “bin/rails about”: ➜ depot git:(master) ✗ bin/rails about | 2019-11-15 | I can't reproduce this | |
10 | SUGGEST | “Since Mac OS X ships with Ruby 2.0.0” , By now lastest macOS Mojave(10.14.6), ships with Ruby 2.3.7p456 . | 2019-10-02 | ||
51 | ERROR | In page 51 when it explains Regular Expressions and then on page 91 when one is used to validate the product model the match end of string, the character sequence provided is \\Z instead of \\z. \\Z is shorthand for \ | 2019-10-02 | ||
91 | ERROR | The validation on page 91 for the image url uses \\Z in the regular expression. This will allow a trailing newline to match the file string. In the product test on page 94 if you add “fred.gif\ | 2019-10-02 | ||
115 | TYPO | Screenshot for page 115 Iteration C3: Using a Helper to Format the Price. The screenshot is cropped so that the change just made with the number_to_currency() helper is not visible. | 2019-10-17 | ||
228 | SUGGEST | In rails6/depot_qb/app/models/order.rb example, we “require ‘pago’ but it seems that Rails isn’t set up to autoload files in the `lib folder. I think we need to add ”rails6/depot_qb/app/models/order.rb" in config/application.rb. Also, using ActiveModel::Serializers::Xml is not mention at any point before this code sample in the book, so perhaps should be removed or explained at some point earlier. | 2019-11-16 | I cannot reproduce the require issue. \n \nThe XML issue is dealt with by another errata so we won't forget that one. | |
437 | ERROR | Not really a technical error nor a typo bu you have the following sentence:- “In Part II, you iteratively built an application, built up test cases along the | 2019-10-17 | ||
178 | TYPO | The book shows the command for updating the line_item table as: Then it shows the output of the migration as : The migration class name doesn’t match the name of the migration in the command, and the output doesn’t match the output for the migration command given. | 2019-10-03 | ||
137 | ERROR | Trying to download the images from media.pragprog.com/titles/rails6/code/rails6/depot_a/app/assets/images/ gives a 403. | 2019-10-03 | ||
128 | OK | Using the params to button_to as stated on this page was insufficient for me. When I tried to add an item to my cart, I got a complaint that I didn’t have the required line_items param. My solution for this was to put the product_id hash inside of another hash with a line_item key so that it matched the generated strong params for the line_items controller. (I feel like I may have done something wrong here to have obtained this error in the first place, but I’m not sure what.) | 2019-11-15 | Can't reproduce. | |
162 | SUGGEST | “Both filenames are treated as a template, executed in Ruby, and sent to the browser as JavaScript for execution. Using .js.erb means you want to write a JavaScript template, and .js.erb means you want to write an ERB template.” This is confusing. Both filenames? What/which filenames are you referring to? .js.erb => Javascript template, and There has to be a cleaner way to express this as I really don’t know what you’re trying to say. And thanks to the guy that posted the correction on page 163. That problem had me scratching my head! | 2019-10-03 | ||
277 | 269 | TYPO | “Note that we want this tag to appear after the call to javascript_include_tag() so The code samples use “javascript_pack_tag” | 2019-10-03 | |
277 | 269 | TYPO | The rails6/depot_t/app/javascript/PayTypeSelector/index.jsx code snippet is missing the arrows where new code has been added for I18n calls | 2019-11-16 | |
132 | TYPO | Screenshot incorrect. A previous commenter noted that it is missing the flash notice, but it is also displaying the wrong route. It should be Your Pragmatic Cart instead of Your Pragmatic Catalog. | 2019-11-15 | This appears to have been fixed in the current beta | |
27 | ERROR | The “brew install chromedriver” has changed. From “brew search chromedriver”: If you meant “chromedriver” specifically: | 2019-10-03 | ||
206 | OK | Missing gem! After the installation of react (going from depot_p to depo_pa) I had to install the gem “activemodel-serializers-xml”, since I got an error that the file was missing. | 2019-11-15 | I can't reproduce | |
159 | TYPO | Screenshot main nav background (excluding the cart) and letters are inverse from style at this point in the book. Main nav should be dark green background (#141) with light text. | 2019-11-15 | This appears to have been fixed since the last beta | |
170 | ERROR | Same issue as noted on 163 where var should be used instead of let or it won’t work the second time. (If this wasn’t fixed in the 10/3/19 fix). | 2019-10-17 | ||
170 | SUGGEST | Comments // START HIGHLIGHT and // END HIGHLIGHT don’t actually match the functionality/purpose of the code between the comments, which is to hide the notice. | 2019-10-17 | ||
173 | ERROR | At the point when you implement the received(data) method of the products_channel.js you have pages other than store#index. If you are visiting any page other than the root URL when when somebody updates a product, your document content will be replaced with the store/index HTML, which is not a good experience if you have another window open in the app. If it’s not something you want to cover in the main chapter, you could add it to playtime. The solution I have below is far from elegant, but solved the problem for now (fragile if you ever add other classes to that element): let mainElement = document.getElementsByTagName(“main”)[0]; | 2019-11-16 | ||
163 | ERROR | In create.js.erb we declare a variable with let. When the “Add to cart” button is clicked, it only updates the cart once. If we click it multiple times we receive an error “Uncaught SyntaxError: Identifier ‘cart’ has already been declared” in the browser console. We can fix this by declaring the variable cart with var instead of let. | 2019-11-16 | ||
180 | TYPO | This could be a typo: Later in the page the file path is correct but the file name turns plural : | 2019-11-15 | ||
163 | ERROR | Hello, example: I hope this will be usefull to the next editions of the book. | 2019-11-16 | ||
215 | SUGGEST | Strange sentence in line3. Should it not be: … the object of that function - which.. | 2019-11-15 | ||
226 | SUGGEST | According your own code examples should line 2 not be: | 2019-11-15 | ||
177 | ERROR | Migration error | 2019-11-15 | I cannot reproduce this | |
206 | ERROR | Again about this missing gem. Sam Ruby wrote: can’t reproduce. But I found the gem “activemodel-serializers-xml” is just in the Gemfile of depot_pa. I was following the book and it is nowhere mentioned this gem should be added. I refer to the issue #85720 pdf page 228 | 2019-11-16 | ||
44 | SUGGEST | In the section for the Mac install, a few suggestions to consider… Catalina has changed from bash to zsh so the echo commands to .bash_profile is not correct if the user is using zsh. This book appears to be usable by very new fresh users. New users to Mac will get zsh on Catalina. Existing users will stay with bash unless they decide to change. Anyhow… its a complication that you might want to think about. Also, after the “rbenv global 2.6.5” command, the path might need to be rehashed. I actually don’t know the best way to do this with zsh. I used “hash -rf” but there may be a better way. With bash, I believe “hash ruby” will work. Again, for very inexperience users, this could trip them up and frustrate them. | 2019-11-16 | ||
234 | SUGGEST | models/order.rb works fine without: | 2019-11-16 | ||
219 | TYPO | Working with depot_t and trying to pay with “Credit Card” and “Purchase Order” it appeared that the paytype_params in the charge! method models/order.rb were nil. | 2019-11-16 | ||
285 | ERROR | In order to make locale_switch.js effective you have to add <%= javascript_pack_tag(‘locale_switcher’) %> In fact you must do it to all the sub-pages yielded in main in application.html.erb. Maybe there is a simpler way. I don’t see it. | 2019-11-16 | ||
123 | TYPO | Oh dear… this is really confusing. At the end of Iteration C1, there is a screen shot of the store index. The screen shot in the PDF is correct and shows Chatbot, Docker, and Crystal. The same screen shot in my epub is wrong and appears to have seeds from depot_ua. | 2019-11-16 | ||
295 | 289 | TYPO | Charter17 should be“Task L,Iteration L1 ,IterationL2 ,IterationL3” ,not J used at Charter15 | 2019-11-15 | |
182 | SUGGEST | After the change to the stylesheet in Iteration F3, the page must be reloaded to get the highlight to work. Just clicking the Add Cart didn’t work. | 2019-11-15 | ||
188 | TYPO | The book has: “Fortunately, Rails generated an outline of what you need to do in app/assets/javascripts/channels/product_channel.js.” The “/assets” directory is not correct. The path (as stated later on) is just “app/javascript/channels/products_channel.js” | 2019-11-15 | ||
189 | SUGGEST | This is for the end of Iteration F5. When the products are updated and broadcast, they are in a different order so instead of just the price changing, they are all redrawn and in a different order. The @products = … statement (I guess) needs to sort them in the same way as the store#index method does but that sorta sucks. | 2019-11-16 | ||
20 | SUGGEST | The suggestion of using a virtual machine is not flushed out well enough. I did not want to install Chrome and it is needed by ChromeDriver. That dependency is not mentioned. There is a chance that I can use Brave which I’m about to explore but rather than do that, I thought it would be simple to switch to a virtual machine. But the rails-dev-box has the same problem. Not only is Chrome not installed but there is no X server or desktop configured. Also, there seems to be an assumption that the virtual machine will be used on Linux but that isn’t really true. I would suggest at least outlining the requirement for Chrome (or perhaps Brave) before ChromeDriver can be used. And I would outline the limitations and assumptions of the rails-dev-box path. | 2019-11-17 | We've combined the virtual machine and linux setup and updated it so that it works. We aren't going to put anything about avoiding Chrome as that is not super common and not a use-case Rails supports. I understand not wanting Google in your life, but Rails system tests are designed to use chrome and chromedriver, so that's what the book will cover. | |
230 | SUGGEST | I got the test:system to pass. In my repo, I had test/system/carts_test.rb which was created during D1 as well as test/system/line_items_test.rb which was created during Iteration D2. These had tests that would not pass. I commented out the tests. The repo under depot_q did not have these files. Also, if anyone cares, I got the ChromeDriver to work with Brave (on a Mac). I did this by creating a file config/initializers/webdrivers.rb with two lines: Webdrivers::Chromedriver.required_version = ‘78.0.3904.70’ To find the first value, start Brave, do “About” and look at the “Chromium” version. Then go to this page chromedriver.storage.googleapis.com/index.html and find the version that is the closet match. Only the first two or three pieces matter. In my case, I matched the first three pieces. Put the string (version) from the web site as the value for required_version. This way, you don’t need to subject your Mac to the abuses of Google. | 2019-11-17 | ||
52 | TYPO | We constructed a toy application…. it should be demo not a toy application. | 2019-11-15 | We are OK with calling it a "toy application" | |
11 | SUGGEST | The macOS installation instructions currently assume bash as the default shell, but in macOS 10.15 that default has been changed to zsh. As such, the book should probably also include instructions on how to set up rbenv with zsh. | 2019-11-16 | ||
245 | ERROR | On page 245 of the PDF when formatting the user form you state "To improve the appearance of the page, we add | 2019-11-17 | ||
226 | ERROR | On pages 226 and 227 of the PDF when configuring email you refer to some code that does not exist in the various environments as follows Depot::Application.configure do This should read Rails.application.configure I’m sure I have reported this before on previous versions of the PDF | 2020-01-03 | This looks fixed in the latest beta | |
226 | 219 | ERROR | The if tests in pay_type_params method use “Credit Card” and “Purchase Order” (2nd word is capitalized). But everywhere else in the code, it is “Credit card” and “Purchase order” (2nd word is all lower case). I’ve just started tracking all this down. I’m not sure why the tests are passing. But when I got to the Active Job (Iteration I2), this line: month,year = pay_type_params[:expiration_date].split(//) was failing because the hash was empty because none of the if conditions were firing because pay_type was “Credit card” but the test was for “Credit Card”. | 2020-01-03 | |
54 | ERROR | The book show a Hash containing instruments: inst_section = { It then shows how to access the values for some keys: inst_section[:oboe] #=> ‘woodwind’ The hash rocket notation for the return value should be # => :woodwind, i.e. a Symbol, not a String. | 2020-01-03 | ||
488 | SUGGEST | Page 488 in my epub (on MacOS). In Iteration K1, the root route changed and added a “via: :all” which is not explained in the text. Also, somewhere in Iteration J2, bin/rails test:system started to fail in the users system test. | 2020-01-03 | ||
285 | ERROR | I don’t know how it should be done but to get locale_switcher to not show the submit button, I did: <%= javascript_pack_tag ‘application’, ‘locale_switcher’, ‘data-turbolinks-track’: ‘reload’ %> in app/views/layouts/application.html.erb. Something seems to be broken because the documentation says javascript_pack_tag “By default, | 2020-01-03 | ||
300 | SUGGEST | Chapter 17, end of Iteration 1, this might be obvious to everyone but me but the “from” field in the test email that you are sending needs to match the route that was created: support@example.com Otherwise you get a routing error and a big ugly stack trace. | 2020-01-03 | ||
xi | TYPO | Some words seem to be missing in the 5th paragraph after “Rails 6le“: “This book covers Rails 6le some of the commands you’ll be using are new, the underlying development model remains the same.“ | 2020-01-03 | ||
189 | ERROR | In the code sample for the LineItem class, :order and :product are being marked as optional, and :cart as required. This should be :order and :cart as optional, and :product as required. | 2020-01-03 | ||
128 | TYPO | The screenshot on this page shows the content of the webpage without any of the styles applied. | 2020-01-03 | I'm not able to see this. On the 128th page of the PDF is a screenshot showing the result of the Rails scaffold which is intentionally unstyled. On the page with the pager number 128 there is no screenshot. | |
252 | ERROR | System tests in chapter15 gives a lot of errors in test_helper.rb: in users_test.rb must be added to setup do: login_as @user ‘Password Confirmation’ also you must not use @user In de Create user test in orders_test : Furthermore I get errors in paytype check in orders_test, not solved yet | 2020-01-03 | Some of this can be dealt with by using the same setup method as we did for the functional tests. I've updated the text in the book to reflect this. The users_test.rb is still broken because that was scaffolded. I've updated the book to call this out and leave it as an exercise (sorry it's too late in the production process to fix it more directly). | |
252 | ERROR | Merry Christmas! Yet forgotten in /system/users_test.rb in create, update and destroy | 2020-01-03 | ||
287 | ERROR | “Task M: Receive Emails and Respond with Rich Text” follows “Task K: Internationalization.” | 2020-01-03 | ||
326 | SUGGEST | It seems nobody ever looked at chapter 19 (Active record) since edition Rails 4. | 2020-01-08 | All references to Fixnum have been removed, approximately six references to pay_type as an string were updated (and pay_type is now used as an example of the before_type_cast being different). Command outputs have been updated. | |
433 | TYPO | bin/rails generate spec:model Cart | |||
14 | SUGGEST | You install google-chrome-stable without a specified version, but for chrome driver, you use version 2.41, so I think they could be incompatible. (The version is 79 for me at 2020/01/30) | |||
4 | ERROR | Chapter 1 - Installing Rails Text indicates Ruby 2.6.0 but graphic indicates Ruby 2.3.3. | |||
5 | ERROR | This entire section needs to be updated. The Windows installer now includes the MSYS2 installation as as final step and the commands in the PDF book are no longer needed. Also, there’s no need to override the extract destination as suggested by the PDF book. Feel free to contact me: r_scott.shaffer@comcast.net for help with installing on Windows. | |||
9 | ERROR | After installing rbenv, running ruby -v returns an error ‘ruby is not installed’ Fix: run rbenv rehash | |||
179 | ERROR | After the form example, the paragraph states: “the first argument, module:order…” fix: should be model | |||
185 | ERROR | In the line_item model code the arrows showing the lines to modify are on the wrong lines. Fix: They should be on the first and the third lines. | |||
200 | ERROR | In the pay_type.jsx file the PayTypeSelector import does not work fix: Add ‘./’ in front: import PayTypeSelector from ‘./PayTypeSelector’ | |||
213 | ERROR | Failed to find chrome binary when running “rails test:system” fix: Install chrome | |||
280 | ERROR | In application.html.erb, the form defaults to POST which breaks. fix: Add explicit method: :get to form tag. | |||
290 | TYPO | Instructions to use support@example.com as the “From” email fix: should be the “To” email | |||
431 | TYPO | command bin/rails generate spec:model Cart does not work fix: should be bin/rails generate rspec:model Cart | |||
317 | SUGGEST | Original text: They’re surprised that they call a model class Person and Rails somehow knows to go looking for a database table called people. wouldn’t it be better to associate Person with the database table called persons? | |||
147 | ERROR | the scss has an error: rails6/depot_i/app/assets/stylesheets/carts.scss you sadly used th instead of td as selector two times. its: should be : | |||
179 | TYPO | The paragraph right below the code snippet about form_with(), there is a typo in “The first argument, module: | |||
23 | TYPO | On p.23, it says “As the second line of the startup tracing indicates…” | |||
183 | SUGGEST | The picture in page 183 shows the effects of the new styles added to carts.scss (in | |||
207 | TYPO | <select id=“pay_type” Based on examples earlier (p. 202) and later (p. 208) this id should be “order_pay_type”. To be fair, the book says “our code will look something like this” so it’s not telling the user to change it. Just makes sense to keep the examples consistent, other things being equal. | |||
200 | ERROR | The ReactDOM.render( document.addEventListener(‘turbolinks:load’, function() { If you check for the presence of that element in the DOM before attempting to render, the errors go away: document.addEventListener(‘turbolinks:load’, function() { | |||
22 | ERROR | In trying to launch the “Hello, Rails” application I keep running into the same issue when navigating to the “work” directory as instructed in the book. Error message reads: “Can’t initialize a new Rails application within the directory of another, please change to a non-Rails directory first.” | |||
12 | ERROR | libreadline-dev is listed twice | |||
364 | TYPO | resources :products, concern: :reviewable should be resources :products, concerns: :reviewable | |||
302 | ERROR | original text: | |||
298 | ERROR | original: No associated order Should be: No associated order | |||
155 | TYPO | "Note that we’ve given the element that wraps the cart the CSS class carts." The element given the css class “carts” was a tag. The was not given a css class, and is not in the preceding code example. You find it back in the _cart.html.erb partial as a basic tag. | |||
167 | SUGGEST | The formatting for this code block is different than all others in the book. The erb <% %> tags have the brackets as light grey. | |||
180 | TYPO | In the screenshot, an erb tag is mistakenly written as “<$= form_with”. It should be “<%= form_with” | |||
212 | ERROR | It is suggested that ChromeDriver is a version of Chrome. It is not. It is used to automate Chrome. | |||
397 | TYPO | “on page 167, we created such a helper method named hidden_div_if” This method was called render_if | |||
109 | ERROR | In the pdf you state According to the rails repository on github there is an important note in the documentation found at github rails sass-rails#important-note Sprockets provides some directives that are placed inside of comments called require, require_tree, and require_self. DO NOT USE THEM IN YOUR SASS/SCSS FILES. They are very primitive and do not work well with Sass files. Instead, use Sass’s native @import directive which sass-rails has customized to integrate with the conventions of your Rails projects. The book should be updated to fully describe the other actions needed when renaming the application.css to application.scss |