Make Field Inheritance a soft dependency

Created on 22 May 2024, about 1 month ago
Updated 11 June 2024, 17 days ago

Problem/Motivation

Currently we have a hard dependency on the Field Inheritance module, but this module is not strictly required for working with events.

The main purpose of having Field Inheritance is to expose content of event series in the view modes of event instances. This is helpful in traditional projects that use Drupal to render the frontend.

On the other hand, projects that adopt a decoupled architecture do not use Drupal view modes for rendering the content. Field Inheritance module does not serve any purpose for these sites and can be disabled.

Proposed resolution

Make the module a recommendation rather than a hard dependency.

✨ Feature request
Status

Closed: works as designed

Version

2.0

Component

Recurring Events (Main module)

Created by

πŸ‡§πŸ‡¬Bulgaria pfrenssen Sofia

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @pfrenssen
  • πŸ‡§πŸ‡¬Bulgaria pfrenssen Sofia
  • πŸ‡ΊπŸ‡ΈUnited States owenbush Denver, CO

    Unfortunately, I don't think this would be possible.

    The title, and description fields on the Event Instance entities are actually inherited from the Event Series entity. If you removed the dependency on Field Inheritance you'd end up with Event Instance entities without titles/labels. So in JSON:API you'd likely end up either with no data for titles for events, or potentially even failures if the code expects a label key to be present.

  • Status changed to Closed: works as designed 17 days ago
  • πŸ‡§πŸ‡¬Bulgaria pfrenssen Sofia

    Thanks for the response. I'm using GraphQL which makes it easy to provide data from both the event instance and its parent eventseries entity in a single query result, so my project works fine without Field Inheritance, but I understand your reasoning. I just wanted to float the idea.

    I'll close this for now.

Production build 0.69.0 2024