Skip to content

Contentable Delegated Type - #119

Merged
A-Mozeak merged 7 commits into
mainfrom
am/107-contentable-delegated-type
Aug 30, 2026
Merged

Contentable Delegated Type#119
A-Mozeak merged 7 commits into
mainfrom
am/107-contentable-delegated-type

Conversation

@A-Mozeak

Copy link
Copy Markdown
Collaborator

Resolves #107

Description

This PR adds a Contentable delegated type that allows Games, Links, and future items to be attached to a ContentModule.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • test/controllers/content_modules_controller_test.rb
  • test/controllers/games_controller_test.rb
  • test/models/content_module_test.rb
  • test/models/content_test.rb

@A-Mozeak

Copy link
Copy Markdown
Collaborator Author

@sean-dickinson mentioned that it'd be good to move Content#destroy_owned_record into Contentable, making it possible to give the assorted content types a default behavior when destroying a ContentModule. I agree, and we'll lean false (i.e. nullable) for the default behavior.

@A-Mozeak
A-Mozeak merged commit 7489aa5 into main Aug 30, 2026
5 checks passed
@A-Mozeak
A-Mozeak deleted the am/107-contentable-delegated-type branch August 30, 2026 13:58
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.

Delegated Types for Content Module Items

2 participants