-
Notifications
You must be signed in to change notification settings - Fork 2
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
TWE-7 - BE - Division page #324
TWE-7 - BE - Division page #324
Conversation
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.
Left a few questions and suggestions. Love the use of factoryboy post generation features.
I am missing the wider project context -- what about the blocks under partner logos ("what we do", "beliefs", "sparks & kindling", "lighting little fires")?
...tyleguide/templates/patterns/_pattern_library_only/streamfield/division_story_container.html
Outdated
Show resolved
Hide resolved
tbx/project_styleguide/templates/patterns/molecules/streamfield/stream_block_division.html
Outdated
Show resolved
Hide resolved
@zerolab Ah sorry I forgot to mention, those are covered by other tickets 😄 Thanks for checking! And thank you for the review~ |
...tyleguide/templates/patterns/_pattern_library_only/streamfield/division_story_container.html
Show resolved
Hide resolved
And creation of HomePage under the Root Wagtail Page
TWE-8 | FE | Division page
Link to Ticket
Description of Changes Made
Equivalent designs for blocks
Dynamic hero block
Introduction section block
Four photo collage block
Other changes:
PartnersBlock
fromservices.blocks
tocore.blocks
so we can reuse it.HomePageFactory
andStandardPageFactory
unit tests with one unit test that checks that all pages have a factory.How to Test
Screenshots
Editor
MR Checklist
Unit tests
Documentation
Browser testing
Data protection
Light and dark mode
Accessibility
Sustainability
Pattern library