Skip to content

Conversation

@goran1010
Copy link
Contributor

@goran1010 goran1010 commented Nov 29, 2025

Because

Fixing lint issues in the Rails Basics section.

This PR

  • Controllers - Add #### Whitelisting "safe" attributes in Rails 3 and #### Handling strong parameters before Rails 8 headings and minor formatting fixes
  • Routing - Add #### Don't use the Postman Chrome Extension heading and minor formatting fixes
  • Views - Address all the MD004, MD005, MD032, TOP004 and many other rule violations, including few minor formatting issues

Issue

Closes #30446

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added the Content: Ruby on Rails Involves the Ruby on Rails course label Nov 29, 2025
@mao-sz mao-sz requested review from a team and zachmmeyer and removed request for a team December 2, 2025 00:30
Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for making these changes!

@JoshDevHub JoshDevHub removed the request for review from zachmmeyer December 2, 2025 20:45
@JoshDevHub JoshDevHub merged commit 26e4653 into TheOdinProject:main Dec 2, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content: Ruby on Rails Involves the Ruby on Rails course

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ruby on Rails: Address lint issues in the Rails Basics section

2 participants