Skip to content

Revert "[dataengines/weather] Add country code to wetter.com location display name"

Bharadwaj Raju requested to merge revert-bc0e8bd7 into master

This reverts commit bc0e8bd7.

The approach used doesn't account for localization differences, and generally we shouldn't mess with the display string as given by the provider.

The change overall doesn't seem to solve a real problem (who has weather widgets of two cities in different countries with the same name?). I suggest we revert it.

Maybe we can add a tooltip with the full station name to remove even such rare ambiguity, but that's for another MR.

cc @nicolasfella @ngraham

Merge request reports