Add Generic and Site settings models. Fix #395 #419
Conversation
| @@ -1444,6 +1365,13 @@ | |||
| "workflow": 1 | |||
| } | |||
| }, | |||
| { | |||
| "model": "wagtailcore.workflowcontenttype", | |||
There was a problem hiding this comment.
From what I can tell this is a new model from a minor wagtail release that just hasn't been included until now
| @@ -3587,6 +3515,22 @@ | |||
| "file_hash": "" | |||
| } | |||
| }, | |||
| { | |||
| "model": "wagtailusers.userprofile", | |||
There was a problem hiding this comment.
Not sure why this was added, doesn't seem to break anything but can't figure out how I managed to add it in either.
There was a problem hiding this comment.
It seems that for some reason we didn’t have a Wagtail user profile instance in the fixtures until now. Perhaps this gets created only when accessed the first time – which we now do on every page as part of determining which theme to show the UI in.
There was a problem hiding this comment.
All looking great @Morsey187! The fixtures changes surprise me a bit but yeah we don’t update the fixtures that often so I’ll probs just split the unrelated changes to a different commit and move on.
thibaudcolas
force-pushed
the
feature/add-site-and-generic-settings
branch
from
4c5571c
to
d9e8ab3
Compare
thibaudcolas
changed the title
|
Thanks @Morsey187 and @thibaudcolas - this is a really helpful addition |
Resolves issue: 395
Adds:
SiteSettingsextending from Wagtail'sBaseSiteSettingclasstitle_suffixwhich is used to replace static text in the base.html'stitle_suffixblockGenericSettingsextending from Wagtail'sBaseGenericSettingclassgithub_url,twitter_url,organisation_urlwhich are now used to conditionally display the footer icons and populate their href attributes.bakerydemo.jsoni.e.load_initial_data