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

feat: Modern footer #448

Merged
merged 11 commits into from
May 27, 2024
Merged

feat: Modern footer #448

merged 11 commits into from
May 27, 2024

Conversation

mgax and others added 11 commits April 24, 2024 13:58
* Add wagtail-factories dependency

* Refactor and extend existing tests

* More tests for ietf.forms

* Skip blocks that are impractical to test

* Tests for bibliography

* Tests for cache purging

* Tests for megamenu

* Blog and IESG statement filtering

* Tests for snippets

* Remove unnecessary OrderedSet implementation

* Remove unnecessary `HomePageBase` class

* Tests for search and 500 page

* Convert tests to pytest format

* Tests for IAB pages

* Normalize tests to pytest style

* Add comment to bibliography extraction code

* Use `LazyAttribute` to generate `slug` values

* Add factory fakers for required fields

* Import `signal_handlers` inside `ready()`

* Better documentation

* More comments

---------

Co-authored-by: Kesara Rathnayake <[email protected]>
* feat: Styling for blockquotes

* Allow empty cells in typed tables

* Render empty float blocks as blank

* Fix migrations
* Extract StructValue class for LinkBlock

* Add footer columns

* Move social icons to footer

* Preview in Wagtail snippet editor

* Tests for the footer

* Responsive styling

* Make the footer expansion accessible

* Make tests work after rebasing onto `main` branch

* Add wagtail-factories dependency

* Improve accessibility

* Add some comments

---------

Co-authored-by: Kesara Rathnayake <[email protected]>
* feat: Styling for blockquotes

* Adjust colours

---------

Co-authored-by: Kesara Rathnayake <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 99.82238% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 96.08%. Comparing base (ec2d7a8) to head (e368cf2).
Report is 8 commits behind head on main.

Files Patch % Lines
ietf/utils/blocks.py 94.44% 1 Missing ⚠️
ietf/utils/models.py 92.85% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #448       +/-   ##
===========================================
+ Coverage   82.81%   96.08%   +13.27%     
===========================================
  Files         126      153       +27     
  Lines        2612     3293      +681     
===========================================
+ Hits         2163     3164     +1001     
+ Misses        449      129      -320     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@microamp microamp self-requested a review May 27, 2024 22:16
@kesara kesara merged commit 271340f into main May 27, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment