-
Notifications
You must be signed in to change notification settings - Fork 62
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
Extra slash added for env_config['cp_url'] #14
Comments
Yes, that's a good catch. We'll tweak that for the future. Thank you! Erik |
Downloaded ee-master-config today - looks good! I notice this bug hasn't been fixed in the repo though. Easy enough to change manually myself but just wanted to bump this issue! |
Ah, thanks for the catch @agemmell. I see there's a pull request about this so we'll merge it in now. Erik |
Fix double slash issue from issue #14
Hello, I just came across the double slash issue today, when troubleshooting two EE add-ons. I removed the trailing slash manually and it works fine now. Just wanted to let you know as it still in the repo. Great setup by the way. Dany |
Hello,
I just noticed that the $env_config['cp_url'] variables gets an extra slash. This is in config.master.php, around like 105. See:
It's safe to remove the slash that appears before admin.php, yes?
The text was updated successfully, but these errors were encountered: