wheelmap-classic icon indicating copy to clipboard operation
wheelmap-classic copied to clipboard

redirect Terms & Privacy URL currently used in iOS app 3.0.2

Open holgerd opened this issue 8 years ago • 9 comments

In the iOS app Version 3.0.2 (build 201) two "old" links still exist:

  1. http://blog.wheelmap.org/was-ist-wheelmap/terms/
  2. This currently redirects to https://community.wheelmap.org/impressum/nutzungsbedingungen/
  3. But it should redirect to https://news.wheelmap.org/nutzungsbedingungen/
  4. blog.wheelmap.org/was-ist-wheelmap/privacy/
  5. This currently redirects to https://community.wheelmap.org/en/imprint/privacy/
  6. But it should redirect to https://news.wheelmap.org/datenschutz/

holgerd avatar Sep 09 '16 11:09 holgerd

cc @Svenyo

holgerd avatar Sep 09 '16 11:09 holgerd

shouldn't those links be fixed in the iphone app?

Xylakant avatar Sep 09 '16 11:09 Xylakant

yes they will. But we need redirects as well for those using the current app version.

holgerd avatar Sep 09 '16 19:09 holgerd

ok, so the best would be to redirect these urls at the cloudflare level, but we've run out of page rules. so either we save some page rules (see for example #386) or we buy some more.

The alternative would be to redirect at the webserver level, which is less efficient. This would require four changes:

  • [x] redirect http://blog.wheelmap.org/was-ist-wheelmap/terms/ to https://news.wheelmap.org/nutzungsbedingungen/ in the old prod/new prod environment
  • [ ] redirect http://blog.wheelmap.org/was-ist-wheelmap/privacy/ to https://news.wheelmap.org/datenschutz/ in the old prod/new prod environment
  • [x] redirect https://community.wheelmap.org/impressum/nutzungsbedingungen/ to https://news.wheelmap.org/nutzungsbedingungen/ in the wordpress blog.
  • [x] redirect https://community.wheelmap.org/en/imprint/privacy/ to https://news.wheelmap.org/datenschutz/ in the wordpress blog.

Xylakant avatar Sep 26 '16 11:09 Xylakant

Yes we need this.

@1000miles can you do this?

holgerd avatar Sep 26 '16 13:09 holgerd

yep

100Miles avatar Sep 26 '16 14:09 100Miles

@holgerd do what? There are two alternatives on the table. The second alternative actually requires the wordpress team to do some of the changes.

Xylakant avatar Sep 26 '16 15:09 Xylakant

@Xylakant imho we should redirect at the webserver level. Since the old iPhone app is already "out there in the wild" we cannot change the URL there.

holgerd avatar Jan 02 '17 13:01 holgerd

Currently two of the redirects listed already happen.

The redirect " redirect https://community.wheelmap.org/en/imprint/privacy/ to https://news.wheelmap.org/datenschutz/ in the wordpress blog." seems to me to be not right. Currently it directs you to https://news.wheelmap.org/en/privacy/ which seems correct. I'm marking that also done.

http://blog.wheelmap.org/was-ist-wheelmap/privacy/ currently redirects to https://news.wheelmap.org/en/privacy/ which is incorrect, should be https://news.wheelmap.org/datenschutz/ .

@schultyy could you modify the nginx file on production? /etc/nginx/sites-enabled/blog.wheelmap.org.

  location /was-ist-wheelmap/ {
-    rewrite ^/was-ist-wheelmap/terms/(.*)$ http://community.wheelmap.org/nutzungsbedingungen/$1 permanent;
+    rewrite ^/was-ist-wheelmap/terms/(.*)$ http://news.wheelmap.org/nutzungsbedingungen/$1 permanent;
+    rewrite ^/was-ist-wheelmap/privacy/(.*)$ http://news.wheelmap.org//datenschutz/$1 permanent;
    rewrite ^/was-ist-wheelmap/haeufig-gestellte-fragen/(.*)$ http://community.wheelmap.org/about/faqs/ permanent;
    rewrite ^/was-ist-wheelmap/partner/(.*)$ http://community.wheelmap.org/about/unterstutzer-partner/ permanent;
    rewrite ^/was-ist-wheelmap/(.*)$ http://community.wheelmap.org/about/$1 permanent;
  }

Hoverbear avatar Jan 19 '17 10:01 Hoverbear