Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.There's a few things that could be improved about the session content type:
- The options for the session Audience are out of order
- The Schedule Info should also have Cancelled or Withdrawn and Backup
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | session-field-options-1702104-8.patch | 4.61 KB | dsdeiz |
| #4 | session-field-options-1702104-4.patch | 6.59 KB | dsdeiz |
| #2 | session-field-options-1702104-2.patch | 4.91 KB | dsdeiz |
Comments
Comment #1
cafuego CreditAttribution: cafuego commentedTagging.
Comment #2
dsdeiz CreditAttribution: dsdeiz commentedI wonder if these are the expected changes:
Comment #3
cafuego CreditAttribution: cafuego commentedOh sorry, I highlighted my suggestions wrong because I added an invalid close tag :-(
I think "Cancelled" or "Withdrawn" (one or the other) and "Backup" should be separate statuses.
Comment #4
dsdeiz CreditAttribution: dsdeiz commentedPatch attached.
Comment #5
sheldonkreger CreditAttribution: sheldonkreger commentedDoes #2 have any dependencies that aren't in the main COD Support branch by default?
Comment #6
ezra-g CreditAttribution: ezra-g commentedWhy would we have a single status for "Cancelled or Withdrawn and Backup" instead of 2 different statuses: "Speaker(s) Cancelled" and "Backup session"?
Seems that if someone definitively cancelled, you'd want to indicate that separately from the session being a backup session.
Comment #7
twardnw CreditAttribution: twardnw commentedSee #3
Comment #8
dsdeiz CreditAttribution: dsdeiz commentedHi, patch attached. Added the statuses "Speaker(s) Cancelled" and "Backup session".
Comment #9
dsdeiz CreditAttribution: dsdeiz commentedComment #10
carsonblack CreditAttribution: carsonblack commented#8 did not seem to have any effect on the "Experience Level" order.
Comment #11
carsonblack CreditAttribution: carsonblack commentedThe patch in #8 must be applied before the cod_session feature is enabled. If you do it after the session is enabled and then revert it the Experience Level Order will not be re-arranged.
This patch is ready to be commited because it works on a fresh install.
Comment #12
japerryIt'd be great to have these options added.
Comment #15
japerryFixed! Restructured the field options to use text keys instead of numerical values. This should be much more scalable if others want to add their own field options. Also added update scripts so views should reflect the new values.