Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Domain Name based i18n #43

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Domain Name based i18n #43

wants to merge 3 commits into from

Conversation

benxao
Copy link

@benxao benxao commented May 28, 2013

have several domain names for the same website because the domain names are localised, allow refinery cms to pick locale base on domain name.

ex: www.iam-on-diet.com , www.je-fais-le-regime.fr

@ugisozols
Copy link
Member

I like the idea but I'm not sure if this should be included by default.

How about putting this PR and refinery/refinerycms#2280 in a separate gem which would add this domain functionality to existing Refinery app. WDYT about this approach?

@simi
Copy link
Member

simi commented May 28, 2013

@ugisozols but than you'll need to override around_recognize :(

@ugisozols
Copy link
Member

@simi yes. Is there a reason it shouldn't be done that way?

@ghost ghost assigned ugisozols May 29, 2013
@benxao
Copy link
Author

benxao commented Jun 3, 2013

@ugisozols I think such a feature is essential to i18n. it is only a few lines of code. Maybe the way I do it is not perfect because I am new too all this. Let me know more about the details.

@ugisozols
Copy link
Member

@benxao I agree that this feature is essential to i18n but I'm just trying to think of (arguably) better way to implement it.

@simi
Copy link
Member

simi commented Jan 22, 2015

@parndt do you think we can close this?

@parndt
Copy link
Member

parndt commented Jan 22, 2015

It's a nice feature

@ugisozols ugisozols removed their assignment Jul 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants