Problem/Motivation

Comment fom Alexander
"In PR #2283 we introduced a new field 'field_event_enable_enrollment' but I can’t from the PR understand what the difference is with 'field_event_enroll?

One is “Enable event enrollment” and the other is “Allow users to enroll in the event” So it appears we duplicated functionality here?

There’s also 'field_enroll_method' to which we could also add a disabled state then we wouldn’t even need `field_event_enroll` anymore either.

Proposed resolution

- Revert PR 2283 entirely
- Include an update hook to remove the new field field_event_enable_enrollment.
- Then fix the original bug with the fields that already existed (field_event_enroll or field_enroll_method).
- Check compatibility with other features (e.g. anonymous enrolment, max event enrollment)

Comments

tBKoT created an issue. See original summary.

ribel’s picture

Title: Revert duplicated functionality » Revert duplicate disable event enrolments functionality
Status: Active » Reviewed & tested by the community

PR was tested and merged and will be included in the 10.3 release

  • ribel committed 5afac5f on 10.3.x authored by tBKoT
    Issue #3232719 by tBKoT: Revert duplicate disable event enrolments...
tbsiqueira’s picture

Status: Reviewed & tested by the community » Fixed
tbsiqueira’s picture

Status: Fixed » Closed (fixed)