ymcatwincities/openy_gated_content

BUG: Virtual Meetings with no description fail to load

Closed this issue · 4 comments

froboy commented

To recreate (tested 14 Sept 2020 in sandbox):

  • Edit a Virtual Meeting
  • Remove the Description
  • Navigate to the Virtual Meetings through the UI and click on the meeting
  • Observe the detail page spins and doesn't load properly.
  • Observe TypeError: this.description is null in the console.

I believe removing .processed in this line would resolve the issue, but I'm not sure if that would break something else.

openy_gated_content/js/gated-content/src/views/VirtualMeetingPage.vue

Line 161 in b125ff5

description: `${this.description.processed}<br>${this.meetingLink.title}: ${this.meetingLink.uri} <br> Virtual meeting page: ${this.pageUrl}`,

FWIW, this behavior does NOT seem to happen with Live Stream content.

Alternatively, the Description field could be required if it should be so.

Virtual_YMCA___Site-Install

froboy commented

Quick visual review of the code looks good to me. Thanks @hamrant!

  • ❤️1
hamrant commented

Fix merged in fivejars fork, will be inclued to next release.
cc @froboy @anpolimus

paige-kiecker commented

Resolved in VY 1.0