6
Happy Labor Day! I labored over a Taylor Swift Acronym Lookup app, and I'd love some feedback
(taylorswiftacronyms.web.app)
General Taylor Swift discussion. Welcome all Swifties!
Bring your clown hats, your wild theories, and your hype, this community is for you.
Some basic guidelines:
Yeah, but it doesn't translate to the site. That's what I'm trying to say :) Your
catch
above doesn't distinguish between 404 or anything else (5xxx) and displaysAn error occurred while processing your request. Please try again later
for all eventualities. So, regardless whether the acronym wasn't found or there was a genuine server error, the same error message is displayed.I honestly have no idea what you had to drag it all out from, but it looks well implemented from the small amount I played around with it. I've never used Firebase, but it looks like you got it working so that's a good job too.
It's probably just my old man brain that saw you were doing this all with files and it felt odd. That's not to say it's wrong, it's just different to what I would have done.
There's a bunch of advantages to databases, like indexes and partial/fuzzy text matching - but I can certainly understand why you went this route if you needed to keep costs down and didn't want to bother with any DB maintenance.
Well done :)
As a fellow old man, at least relative to this fanbase, I fully understand, and this is exactly the kind of feedback I was hoping to get. Thanks!
As a devops engineer, sometimes the most efficient server is the one that doesn't exist; next best is the one that someone else pays for. If heroku free tier existed, I'd consider using that to handle queries server side and aggressively cache them in a CDN.