-
Notifications
You must be signed in to change notification settings - Fork 16
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
Translatable external URL #84
Comments
I had the same problem and to fix it I had to fork the plugin, add a field for internal_url and external_url and change the way the |
@osmanzeki do you have considered to place a PR with your changes? |
@matteotrubini I considered it but as things sometimes are I had a deadline to meet and did not attempt a backward compatible approach. As such I had to add columns and change the way the URL is handled overall. I'll see if I have time to attempt an api compatible PR when I have a minute to work on this. |
@osmanzeki got it, I've the same troubles 😄 |
@matteotrubini yes, as long as there is no breaking changes I'm ok |
CONTEXT:
multilanguage website managed with Rainlab.Translate
NEEDS:
menu items liked to external URL could have different href depending on current locale:
Simply add
external_url
andurl
to$translatable
fieldsoc-menumanager-plugin/models/Menu.php
Line 29 in d7b9ccc
url
attribute plusis_external
flag).Do you think you can support that?
In the meanwhile, the only workaround that I see is to use a CMS page to manage a "redirect router" somehow. Any other ideas?
The text was updated successfully, but these errors were encountered: