Rails Turbo Drive: Add knowledge check section with 7 questions #29862
Conversation
github-actions
bot
added
the
Content: Ruby on Rails
|
It seems that there are a lot of linting errors. May I fix these stuff too? If so, how would I run linters locally instead of having to push everytime and get GitHub to run the checks? |
rlmoser99
requested review from
a team and
linkonsat
and removed request for
a team
Not a maint so not saying you need to fix the lint errors now, but to do linting stuff locally, instructions are in this repo's CONTRIBUTING.md |
Sorry, are you referencing to this link? I really can't find the instructions. https://github.com/TheOdinProject/.github/blob/main/CONTRIBUTING.md |
No, that's the general contributing guide in the |
|
Ah I see, thank you. That's my first time seeing that 🤔. |
There was a problem hiding this comment.
Thank you for making this PR @Monatric
I have a couple recommended changes to the knowledge check links, and while we're here in this file, we should go ahead and fix the Markdown lint errors. Let me know if you have any questions about any of this.
|
|
||
| The following questions are an opportunity to reflect on key topics in this lesson. If you can't answer a question, click on it to review the material, but keep in mind you are not expected to memorize or master this knowledge. | ||
|
|
||
| - [What is page navigation defined by Turbo Drive?](https://www.theodinproject.com/lessons/ruby-on-rails-turbo-drive#page-navigation) |
There was a problem hiding this comment.
| - [What is page navigation defined by Turbo Drive?](https://www.theodinproject.com/lessons/ruby-on-rails-turbo-drive#page-navigation) | |
| - [What is page navigation defined by Turbo Drive?](#page-navigation) |
Section headings automatically have an id inserted for them, which means you can do internal page navigation through just the id.
This isn't just a keystroke saver either. If you were running this site in dev and serving on localhost:3000, you'd want the links to go to localhost:3000/lessons/ruby-on-rails-turbo-drive#page-navigation and not the actual production site. Just expressing that if you can, you want to avoid hardcoding the full production URL to a page (or section of a page).
There was a problem hiding this comment.
Got it! But what's the next thing to do from here? Do I make changes locally or click "Commit suggestion" and "Resolve Conversation"?
I will work on it now but for now I will make the changes locally since I think the results would be the same anyway (i hope)
| - [What is page navigation defined by Turbo Drive?](https://www.theodinproject.com/lessons/ruby-on-rails-turbo-drive#page-navigation) | ||
| - [What is the default visit action that adds a new entry to the browser history?](#visit-action) | ||
| - [How do you change the action of a Turbo Drive link?](#change-turbo-action) | ||
| - [How do you disable Turbo Drive?](https://www.theodinproject.com/lessons/ruby-on-rails-turbo-drive#disable-turbo-drive) |
There was a problem hiding this comment.
| - [How do you disable Turbo Drive?](https://www.theodinproject.com/lessons/ruby-on-rails-turbo-drive#disable-turbo-drive) | |
| - [How do you disable Turbo Drive?](#disable-turbo-drive) |
I think I resolved all lint errors now, though I have this slightly annoying issue once I installed the markdownlint VSCode extension, where if I run the 
|
Because
The Turbo Drive lesson is oddly missing knowledge checks, in contrast with other lessons that do contain them. This PR adds 7 knowledge check questions, with links to the answer, and added some
spantags as necessary. This PR should allow users to reflect more towards link navigation with Turbo.This PR
Issue
Closes #29803
Additional Information
Pull Request Requirements
location of change: brief description of changeformat, e.g.Intro to HTML and CSS lesson: Fix link textBecausesection summarizes the reason for this PRThis PRsection has a bullet point list describing the changes in this PRIssuesection