Skip to content

Fix: enum case #52

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

Merged
merged 6 commits into from
Aug 20, 2024
Merged

Fix: enum case #52

merged 6 commits into from
Aug 20, 2024

Conversation

MrPunyapal
Copy link
Owner

No description provided.

The code changes update the logic for determining the `is_featured` status of a post in the `FeaturedPostController`. The `is_featured` value is now compared against the `FeaturedStatus` enum values `Featured` and `NotFeatured` instead of `FEATURED` and `NOT_FEATURED`. This change aligns with the recent update to the `FeaturedStatus` enum to use camel case for case names.
The code changes update the default value of the `is_featured` column in the `create_posts_table` migration. The default value is changed from `FeaturedStatus::NOT_FEATURED` to `FeaturedStatus::NotFeatured` to align with the recent update to the `FeaturedStatus` enum to use camel case for case names.
The code changes update the enum values in the `FeaturedStatus` enum to use camel case for case names. This aligns with the recent updates made to the `is_featured` logic in the `FeaturedPostController` and the default value in the `create_posts_table` migration.
@MrPunyapal MrPunyapal merged commit e947c85 into main Aug 20, 2024
1 check passed
@MrPunyapal MrPunyapal deleted the fix/enum-case branch August 20, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant