Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Update Migros-Bank.xml - #14340

Merged
jeremyn merged 3 commits into
EFForg:masterfrom
cschanaj:Migros-Bank.xml
Jan 19, 2018
Merged

Update Migros-Bank.xml#14340
jeremyn merged 3 commits into
EFForg:masterfrom
cschanaj:Migros-Bank.xml

Conversation

@cschanaj

Copy link
Copy Markdown
Collaborator

No description provided.

banking.migrosbank.ch is gone from DNS
@jeremyn

jeremyn commented Jan 19, 2018

Copy link
Copy Markdown
Contributor

Removing banking is fine.

The blog behavior isn't right though. Without browser history, and not counting intermediate redirects, the redirects work like this:

http://blog.migrosbank.ch > https://blog.migrosbank.ch/de/
http://blog.bancamigros.ch > https://blog.migrosbank.ch/it/
http://blog.banquemigros.ch > https://blog.migrosbank.ch/fr/

Also it looks like once you visit one language site, it can affect what happens with other sites. For example if you first visit http://blog.banquemigros.ch and get redirected to the fr/ site, and then visit http://blog.migrosbank.ch , you'll be redirected to fr/ instead of de/. However http://blog.bancamigros.ch will still redirect you to https://blog.migrosbank.ch/it/ .

I think we should remove the targets for both http://blog.bancamigros.ch and http://blog.banquemigros.ch and their rules.

@jeremyn jeremyn self-assigned this Jan 19, 2018
@cschanaj

Copy link
Copy Markdown
Collaborator Author

@jeremyn travis is passing now. thanks!!

@jeremyn
jeremyn merged commit f7fdd64 into EFForg:master Jan 19, 2018
@jeremyn

jeremyn commented Jan 19, 2018

Copy link
Copy Markdown
Contributor

Thanks, merged.

@jeremyn jeremyn removed their assignment Jan 19, 2018
@cschanaj
cschanaj deleted the Migros-Bank.xml branch January 20, 2018 03:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants