We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The APP_URL configuration setting in Laravel is supposed to include the scheme.
For instance in the .env.example file :
.env.example
APP_URL=https://mydomain.com
See https://github.com/laravel/laravel/blob/master/.env.example for instance and https://github.com/laravel/laravel/blob/master/config/app.php#L55 too.
So in the Twill module controller, getPermalinkBaseUrl() must not add the scheme in front of the APP_URL config of Laravel :
getPermalinkBaseUrl()
https://github.com/area17/twill/blob/master/src/Http/Controllers/Admin/ModuleController.php#L1388
This breaks the permalink href (with a double scheme, such as http://https://mydomain.com)
http://https://mydomain.com
The method should be something like :
return Config::get('app.url') . '/' . ($this->moduleHas('translations') ? '{language}/' : '') . ($this->moduleHas('revisions') ? '{preview}/' : '') . ($this->permalinkBase ?? $this->moduleName) . (isset($this->permalinkBase) && empty($this->permalinkBase) ? '' : '/');
The text was updated successfully, but these errors were encountered:
We're now talking about this line https://github.com/area17/twill/blob/master/src/Http/Controllers/Admin/ModuleController.php#L1500
Sorry, something went wrong.
I guess this is fixed by #651
ifox
No branches or pull requests
The APP_URL configuration setting in Laravel is supposed to include the scheme.
For instance in the
.env.example
file :See https://github.com/laravel/laravel/blob/master/.env.example for instance and https://github.com/laravel/laravel/blob/master/config/app.php#L55 too.
So in the Twill module controller,
getPermalinkBaseUrl()
must not add the scheme in front of the APP_URL config of Laravel :https://github.com/area17/twill/blob/master/src/Http/Controllers/Admin/ModuleController.php#L1388
This breaks the permalink href (with a double scheme, such as
http://https://mydomain.com
)The method should be something like :
The text was updated successfully, but these errors were encountered: