Search title shouldn't be required anymore

news-catalyst / google-app-scripts

a collection of scripts for tinynews google docs add-ons

https://script.google.com/a/newscatalyst.org/d/1ILURq69o3cYUy6k1n1X6HwxdMfl9xWNhILYuZxgLfeblb3IR15WCMZSj/edit

1 stars 1 forks source link

Search title shouldn't be required anymore #396

Open TylerFisher opened 2 years ago

TylerFisher commented 2 years ago

We should just fall back to headline if it's blank, like we do for FB/Twitter

chelsea-tnc commented 2 years ago

It looks like there was a pull request, but it didn't get merged. Was there something blocking?

TylerFisher commented 2 years ago

I think this is one of those things that got written before the Vercel platforms migration, so I couldn't merge it after we made the migration. The idea is that the frontend (next-tinynewsdemo) code is written to fallback to the headline for search titles if people don't provide a search title. However, in the Google Apps Script form, our form validation requires that a search title is provided. We should remove that validation from the form here.