-
Notifications
You must be signed in to change notification settings - Fork 86
template field not saved #414
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
Comments
I've opened a PR here with the changes required #415 |
I can also confirm this bug. Creating a new page is fine, but editing and trying to change the template, it doesn't save the updated template value. |
@ShaunWilders does this fix work for you? #415 |
@ryanmitchell it does. Just tested and the fix resolves the issue for me. For context, we don't run multi-site, although we did set this up and then remove at one point; and we have entries and users in the database, but everything else is file based. |
Hi @ryanmitchell . I came here because I was having the same issue, but the latest PR doesn't help if you set your template field to null. Seems like we should instead check if |
If you have a failing test and a solution in a PR I’ll happily take a look at it. If not I can loop back and take another look. |
Bug description
Cannot save template field.
I dont know if its interested for this but I have a Multi Site Setup.
How to reproduce
Logs
Environment
Additional details
No response
The text was updated successfully, but these errors were encountered: