Update event content type based on content strategy

Created on 26 November 2024, 5 months ago

Problem/Motivation

The event content type needs to be updated to match the content strategy, for alignment with the other content types. It should use the basic page fields, plus add the additional fields:
https://docs.google.com/spreadsheets/d/1im8CByn4Lo9D6WjC-sj_m1uQojaUFmsk...

📌 Task
Status

Active

Component

Track: Event

Created by

🇦🇺Australia pameeela

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

Merge Requests

Comments & Activities

  • Issue created by @pameeela
  • First commit to issue fork.
  • Pipeline finished with Canceled
    5 months ago
    Total: 855s
    #350488
  • Merge request !233Issue #3489781: Update event content type. → (Merged) created by diwakar07
  • 🇮🇳India diwakar07

    Hi,
    I have created an MR for updating the content type.
    @deepali sardana, I was already working on the issue and hence have created an MR. If you wish, you can review the MR provided.

    Moving to Needs Review.
    Please review.
    Thanks.

  • Pipeline finished with Success
    5 months ago
    Total: 1173s
    #350511
  • Pipeline finished with Success
    5 months ago
    Total: 1143s
    #350513
  • 🇺🇸United States phenaproxima Massachusetts

    This is a good start, but it's incomplete. Changes are needed in `recipe.yml` too (and possibly other files). Additionally, there are elements of the content model itself which make little sense:

    • Why is the Tags field for events marked as having a cardinality of 1? (For that matter, why are there separate "Values" and "Cardinality" columns?)
    • What is the purpose of the "File" field? There's no description.
    • The "Link" field is also incredibly vague.

    Assigning to @pameeela to provide answers on these inconsistencies.

  • 🇺🇸United States phenaproxima Massachusetts

    Needs work anyway...

  • 🇺🇸United States phenaproxima Massachusetts

    Note that this will remove the Locations recipe entirely in favor of an Address field on the Event content type.

  • 🇦🇺Australia pameeela

    Sorry all for the confusion and changing info, I've made some updates to the spreadsheet based on various conversations and to answer questions.

    There's a fair bit to do here, in addition to removing the locations recipe we'll need to move all the geolocation stuff over so we can get the map on our event display. I'd be OK to do that in a follow up though if we want to stick to updating the content type here.

  • Pipeline finished with Failed
    5 months ago
    Total: 1088s
    #351893
  • Pipeline finished with Failed
    5 months ago
    Total: 1384s
    #351894
  • 🇺🇸United States phenaproxima Massachusetts

    Self-assigning to merge the Locations stuff into Events.

  • Pipeline finished with Failed
    5 months ago
    Total: 386s
    #351976
  • Pipeline finished with Canceled
    5 months ago
    Total: 175s
    #351982
  • Pipeline finished with Canceled
    5 months ago
    Total: 434s
    #351987
  • Pipeline finished with Canceled
    5 months ago
    Total: 64s
    #351995
  • Pipeline finished with Canceled
    5 months ago
    Total: 486s
    #351997
  • Pipeline finished with Canceled
    5 months ago
    Total: 303s
    #352007
  • Pipeline finished with Canceled
    5 months ago
    Total: 277s
    #352023
  • Pipeline finished with Canceled
    5 months ago
    Total: 202s
    #352034
  • Pipeline finished with Failed
    5 months ago
    Total: 1447s
    #352044
  • 🇺🇸United States phenaproxima Massachusetts

    I think I have this where I want it - the fields match what's in the content strategy, at least in terms of their type. Some of the names are different (body instead of field_content, for example), but that's a problem across all content types anyway.

    I think @pameeela can review this.

  • Pipeline finished with Success
    5 months ago
    Total: 1107s
    #352103
  • 🇺🇸United States phenaproxima Massachusetts
  • 🇺🇸United States phenaproxima Massachusetts
  • Pipeline finished with Failed
    5 months ago
    Total: 1271s
    #352194
  • 🇦🇺Australia pameeela

    This is beyond awesome, I was not expecting the location/map stuff to get sorted!

    Just made some minor tweaks, initially was super confused about why there was still a location node but think I figured that out. Also changed the body to optional, I know the sheet said required but I've changed that now.

  • 🇺🇸United States phenaproxima Massachusetts

    I'll take that as an RTBC.

  • Pipeline finished with Canceled
    5 months ago
    Total: 909s
    #353634
  • Pipeline finished with Failed
    5 months ago
    Total: 916s
    #353633
  • Pipeline finished with Failed
    5 months ago
    Total: 1269s
    #353647
  • Pipeline finished with Skipped
    5 months ago
    #353672
  • 🇺🇸United States phenaproxima Massachusetts

    Merged into 0.x. Thanks everyone!

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024