search

WPTT / theme-sniffer

Theme Sniffer plugin using sniffs.
MIT License
270 stars 3 forks source link

Check Author-URI and Theme-URI availability #119

Open timelsass opened 5 years ago

timelsass commented 5 years ago

Feature Request:

When doing the sniff on author-uri and theme-uri's it would be nice to check if they also load successfully. If they are just 404 we could provide authors with informative text that the URLs must be active/relevant.

dingo-d commented 5 years ago

Seems doable. Just one thing that's a bit worrisome to me is if we want to use theme sniffer on wordpress.org site (like a stand in for current theme check), how much of an impact to performance would that have?

timelsass commented 5 years ago

I don't think it would be very resource intensive done server side. If it would only be done when a user is uploading a zip, I couldn't imagine it being much of an impact at all. We could also pass the data parsed to js and do the request clientside

dingo-d commented 5 years ago

Was this implemented or does a PR exist for this?