samvera / hyku-next

Hyku informational static web site built with NextJS
https://samvera.github.io/hyku-next/
1 stars 0 forks source link

"Online" toggle for Presentations displays incorrect status (Remove) #4

Closed DraxIndustries79 closed 3 months ago

DraxIndustries79 commented 3 months ago

Story

I confess, I'm not sure what the "Online" yes/no option refers to in Presentations. It's either meant to show or hide this entry on the site, or means that the presentation itself is available online. Either way, it doesn't work. It shows "Online" as green regardless of selecting yes, no, or leaving unchecked. I also don't think it's particularly useful and would vote for removing.

The radio button in the Presentations creation UI:

Screenshot 2024-05-28 at 7 34 30 PM

How it displays on https://samvera.github.io/hyku-next/presentations:

Screenshot 2024-05-28 at 7 34 10 PM

Acceptance

adamjarling commented 3 months ago

@DraxIndustries79 Ha, yea that is the intent. I can do either, wire it up so Green if the presentation is online, and have the little thing not appear at all if not online. Or I can just scrap the element all together. Your call?

adamjarling commented 3 months ago

This is updated to only show the green button if the conference "Online" is set to true in Contentful for an individual Presentation.