Skip to content
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

new tile blocks #100

Merged
merged 3 commits into from
Jun 13, 2024
Merged

new tile blocks #100

merged 3 commits into from
Jun 13, 2024

Conversation

sarahframe
Copy link

  • Article tile block
  • Event tile block
  • add featured checkbox to standard tile block

help_text="Define the maximum number of tiles to show in this group.",
)

see_more_link = InternalPageLinkBlock(help_text="'See more' link", required=False)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this ever be anything other than the landing page? Maybe we could just make this control the text?

.get_children()
.live()
.public()
.order_by("-blogpost") # Sort by date descending
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this not first_published_at?

@@ -1,5 +1,9 @@
{% load wagtailcore_tags %}

{% if self.featured %}
{# make it look different }

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

broken comment format (missing #})

@sarahframe sarahframe merged commit 6893743 into main Jun 13, 2024
3 of 5 checks passed
@sarahframe sarahframe deleted the feature/tile-blocks-bed branch June 13, 2024 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants