jupiterbroadcasting.com icon indicating copy to clipboard operation
jupiterbroadcasting.com copied to clipboard

Show contact links - point to ./contact

Open gerbrent opened this issue 3 years ago • 7 comments

Now that we have a contact page thanks to #139, I believe it would be logical to point the Show pages to have the "Contact" link point to https://jupiterbroadcasting.net/contact/ instead of, say, https://coder.show/contact

it would mean:

  • losing the functionality of having the show be auto-chosen for feedback (listeners would have to explicitly choose say Coder Radio from the dropdown menu)
  • improved UX: not having listeners be taken away from the main JB site

I believe that's an acceptable compromise. Thoughts? Open to feedback, as always.

gerbrent avatar Aug 02 '22 15:08 gerbrent

The link on the show page could pass a url query parameter which could be parsed with some JS to automatically fill the form with the right show.

reesericci avatar Aug 05 '22 00:08 reesericci

I think this is even easier with WuFoo. They allow URL pre-loading in their forms. @gerbrent if you can get me the API_IDs I can test this out to see if the URL modification will work for our purposes. Source: https://help.wufoo.com/articles/en_US/kb/URL-Modifications

My idea is that we use the contact link like this: https://jblive.wufoo.com/embed/w7x2r7/def/field1=coder-radio (where field1 is the API_ID). It would be a pretty easy re-write on the Hugo side since that can be dynamically inserted.

Lord0fBytes avatar Aug 05 '22 12:08 Lord0fBytes

LOVE this. Stay tuned....

gerbrent avatar Aug 05 '22 15:08 gerbrent

@Lord0fBytes

if you can get me the API_IDs

You can find them yourself by looking at the source code ;)

The show dropdown has an ID of Field7 and the options are the show names, so Coder Radio would be pre-selected using: https://jblive.wufoo.com/embed/w7x2r7/def/Field7=Coder%20Radio

Having said that, I'm not sure if we're going to keep using Wufoo. Issue #232 is now part JB 1.0.

reclaimingmytime avatar Aug 07 '22 08:08 reclaimingmytime

Awesome I appreciate that info. I'll make the change to it anyways because it's literally two lines of code.

If they go away from it no big deal.

Lord0fBytes avatar Aug 07 '22 11:08 Lord0fBytes

I haven't been keeping up with all the PRs, @Lord0fBytes has there been any progress on this? (Just personally curious how the implementation will work)

elreydetoda avatar Aug 12 '22 16:08 elreydetoda

@Lord0fBytes still wanna work on this one?

gerbrent avatar Oct 11 '22 02:10 gerbrent