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

feat: rails | AR queries | enums (#29099) #29116

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Nov 23, 2024

  1. feat: rails | AR queries | enums (TheOdinProject#29099)

    Add lesson content: section on enums
    
    * use cases
    * benefits
      * readability
      * maintainability
      * speed
      * disk space
    * how to implement
      * model
      * migration
        * warn about changing persisted data after
          change to enum declaration
    * comparison to scopes
    * class/instance methods exposed
    sean-garwood committed Nov 23, 2024
    Configuration menu
    Copy the full SHA
    5a66c96 View commit details
    Browse the repository at this point in the history

Commits on Dec 2, 2024

  1. fix: rails | AR queries | enums (TheOdinProject#29099)

    implement changes suggested during review
    
    * drop array model declaration
    * rearrange lesson content
      * move up migrations
      * move down equivalent scopes
    * use more pertinent example
      * light_switch => article (draft, published)
    * unwrap lines
      * my editor defaulted to 80 char wrapping.
      * unwrapped all lines in enums/modified sections
    * drop mention of/link to discrete values
    * clarify enum definition
      * typically, not always, an int
    sean-garwood committed Dec 2, 2024
    Configuration menu
    Copy the full SHA
    d9af9ec View commit details
    Browse the repository at this point in the history