-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#6598] gruenbuch/models: extend oveview page to create index page with migra… #535
Conversation
help_text=apps_blocks.HELPTEXT_RICHTEXT_A11Y) | ||
page_intro_de_ls = fields.RichTextField( | ||
blank=True, default="", | ||
verbose_name="Grünbuch Overview page introduction", | ||
verbose_name="Page introduction de ls", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so it matches title form fields and can be used generically
208578b
to
3899f9f
Compare
3899f9f
to
36f4d63
Compare
@@ -115,6 +116,15 @@ class GruenbuchOverviewPage(MetadataPageMixin, Page): | |||
] | |||
|
|||
|
|||
class GruenbuchIndexPage(GruenbuchOverviewPage): | |||
template = 'gruenbuch_overview_page.html' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its the same as parent page, right? So do you need that again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when I didn't have it it automatically looked for a template called index_page which didn't exist
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah cool! Even though this structure is confusing, working all fine!
…tion
I don't know if this is the best way to do this?!