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

Chore: Renames has_live_preview column to previewable. #4039

Conversation

ZachBaird
Copy link
Contributor

Because

The new column name is Railsyer.

This PR

  • Adds a migration to rename lesson.has_live_preview to lesson.previewable
  • Changes all instances of has_live_preview to be previewable in the codebase.

Issue

Closes #4003

Additional Information

As a matter of clarification, the task assumed the has_live_preview column was in the project_submissions table. It was actually on the lessons table.

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the keyword: brief description of change format, using one of the following keywords:
  • Feature - adds new or amends existing user-facing behavior
  • Chore - changes that have no user-facing value, refactors, dependency bumps, etc
  • Fix - bug fixes
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • I have verified all tests and linters pass after making these changes.
  • If this PR addresses an open issue, it is linked in the Issue section
  • If applicable, this PR includes new or updated automated tests

@KevinMulhern KevinMulhern temporarily deployed to odin-review-app-pr-4039 July 29, 2023 17:26 Inactive
Copy link
Member

@KevinMulhern KevinMulhern left a comment

Choose a reason for hiding this comment

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

Great stuff, thanks mate 💪

@KevinMulhern KevinMulhern merged commit 6fd725d into TheOdinProject:main Jul 29, 2023
Mclilzee pushed a commit to Mclilzee/theodinproject that referenced this pull request Aug 2, 2023
…t#4039)

## Because
The new column name is Railsyer.


## This PR
- Adds a migration to rename lesson.has_live_preview to
lesson.previewable
- Changes all instances of has_live_preview to be previewable in the
codebase.


## Issue
Closes TheOdinProject#4003 

## Additional Information
As a matter of clarification, the task assumed the has_live_preview
column was in the project_submissions table. It was actually on the
lessons table.


## Pull Request Requirements
- [x] I have thoroughly read and understand [The Odin Project
Contributing
Guide](https://github.com/TheOdinProject/theodinproject/blob/main/CONTRIBUTING.md)
- [x] The title of this PR follows the `keyword: brief description of
change` format, using one of the following keywords:
  - `Feature` - adds new or amends existing user-facing behavior
- `Chore` - changes that have no user-facing value, refactors,
dependency bumps, etc
  - `Fix` - bug fixes
-   [x] The `Because` section summarizes the reason for this PR
- [x] The `This PR` section has a bullet point list describing the
changes in this PR
- [x] I have verified all tests and linters pass after making these
changes.
- [x] If this PR addresses an open issue, it is linked in the `Issue`
section
-   [x] If applicable, this PR includes new or updated automated tests
@ZachBaird ZachBaird deleted the chore/rename-project-submission-previewable branch September 7, 2023 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Chore: Rename project submissions has_live_preview to previewable
2 participants