Accuweather.com's web page states error 500

VERIFIED FIXED

Status

Websites
Adaptive Search
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: nhirata, Assigned: clouserw)

Tracking

Staging

Details

Attachments

(1 attachment)

Created attachment 8628006 [details]
2015-06-30-09-26-30.png

1. add weather collection
2. go to weather
3. launch accuweather

Expected: accuweather website mobile version
Actual: This page has been struck by lightning.

stage commit : 5fcd890da4a2acf345a7a2f1d15fcd974febe452
Note: this happens on fxos 1.x and 2.x ; clicking the mobile site map will go to the mobile version of the site.
(Assignee)

Comment 1

2 years ago
It looks like accuweather changed their site.  There is a link for local detection on the front page so I changed it to that in https://github.com/mozilla/everything.me/commit/a644d7322b0d3e7fe579dd59a4f3703142d0d6cf

However, this was across many files again so I wrote a script again which was sketchier than the last one.  In my spot checks nothing broke, but there is some amount of risk there.  The script was:

> grep -rsl "m.accuweather.com/location/current" | xargs sed -i 's@m.accuweather.com/location/current[^"]*"@m.accuweather.com"@'
Assignee: nobody → wclouser
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
verified fixed on prod
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.