{"push_id":4383831925,"size":21,"distinct_size":1,"ref":"refs/heads/pfc-net-prices","head":"066ff63b3fe07f04baa2cd3ec0ace18b5850c98f","before":"bbe98897d7124fec6b7dbac2f91ebf897ce1463c","commits":[{"sha":"5991617888dceecba0a2f991a46e831b97f1ffa0","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Move new home page carousel content into Wagtail","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/5991617888dceecba0a2f991a46e831b97f1ffa0"},{"sha":"7358034e0c8c5feffacb9ec91d3e539aadba194e","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Migrate new home page content into Wagtail\n\nThis set of changes moves the content for the new home page design\ninto configuration for the Wagtail HomePage page type.\n\nServing the new content from hardcoded values is still supported via the\nexisting NEW_HOME_PAGE feature flag. A new NEW_HOME_PAGE_IN_WAGTAIL\nfeature flag can be used to instead serve the content from Wagtail. If\nboth flags are enabled, the Wagtail one takes precedence.\n\nThere are several changes here related to Wagtail InlinePanels and the\ndjango-modelcluster package that powers them. In the version of Wagtail\nthat we are using (1.13), nested InlinePanels are not supported at all.\nIn versions >2.0, nested InlinePanels are supported via a workaround\ndetailed on wagtail#5511. Full support for nested InlinePanels will be\navailable in Wagtail starting in version 2.8, see wagtail#5566.\n\nIt turns out that we can use the nested InlinePanel workaround on\nWagtail 1.13, but to do so we need to increment the version of\ndjango-modelcluster from 3.1 to 4.4. Unfortunately, Wagtail 1.13\nrequires django-modelcluster<4.0 even though 4.4 still supports it,\nalong with Python 2 (support for both of these is not removed until\ndjango-modelcluster 5.0). So we want to force use of\ndjango-modelcluster==4.4, even though it technically conflicts.\nUnfortunately, our use of tox_pip_extensions_ext_venv_update prevents\nthis (it doesn't allow you to specify any conflicting versions, even\nthough pip does out of the box; see venv-update#105 on this including a\ncomment from an old colleague). So, I've (hopefully temporarily)\ndisabled that until we are on Wagtail 2.\n\nAfter all that you may be wondering why use nested InlinePanels at all.\nThe alternative to this would be to use StreamFields, but this felt like\na worse implementation to me: harder migrations, worse user experience\nin the Wagtail editor, etc. I welcome any feedback on this choice.\n\nThese changes include a new Django management command to automatically\npopulate the Wagtail fields from the hardcoded values. After running\nthe new v1 migration on your database, run\n\ncfgov/manage.py migrate_home_page_content\n\nto update both the English and Spanish pages.","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/7358034e0c8c5feffacb9ec91d3e539aadba194e"},{"sha":"6d5887bfda321253a504797bd4f741aece59af44","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Correct reference to Wagtail 1.13","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/6d5887bfda321253a504797bd4f741aece59af44"},{"sha":"29ac02ad66991ae0256edc89e9d8d54c9f28b648","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Remove unnecessary carousel block","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/29ac02ad66991ae0256edc89e9d8d54c9f28b648"},{"sha":"43a41c33271e84edfd767fff81953536e091f8b0","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Move Configuration panel last","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/43a41c33271e84edfd767fff81953536e091f8b0"},{"sha":"b0a28b56733fb7edba49de9afb76cb517dfe202c","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Fix a typo","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/b0a28b56733fb7edba49de9afb76cb517dfe202c"},{"sha":"b7532e80bfe4d1e9f5446430417893e208079d84","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Merge pull request #5383 from chosak/home-page-carousel-in-wagtail\n\nMove new home page content into Wagtail","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/b7532e80bfe4d1e9f5446430417893e208079d84"},{"sha":"618bb6757000f78132ee184acd96b55b1bf49d19","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"New home page content edits","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/618bb6757000f78132ee184acd96b55b1bf49d19"},{"sha":"7494263bf3d0b518f7328a4293dfd01f2301e129","author":{"name":"Shia LaBot","email":"d16c4ab686498ad36d5e7446f3781c9f5833bed7@users.noreply.github.com"},"message":"Merge pull request #5407 from chosak/home-page-content\n\nNew home page content edits","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/7494263bf3d0b518f7328a4293dfd01f2301e129"},{"sha":"ea2efd4b9bc352983ff53490d6c1c29dcf2456b2","author":{"name":"virginiacc","email":"44b6969d4b7ba6f4589dd981daaa9f42c42147ee@gmail.com"},"message":"Prevent card content from overflowing container on small screen sizes. (#5404)","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/ea2efd4b9bc352983ff53490d6c1c29dcf2456b2"},{"sha":"e2813c0e0b33d7812d9690e75dd09b3e70a40962","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Fix new home page info unit ordering\n\nThis change fixes the order of HomePageInfoUnit class inheritance to fix\nWagtail admin ordering that wasn't working properly.\n\nSee wagtail#4477#issuecomment-384782198.","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/e2813c0e0b33d7812d9690e75dd09b3e70a40962"},{"sha":"288199fca18c34ecb27cc1f7fc9fc84c32ee19d4","author":{"name":"Andy Chosak","email":"35806a203e6fdda38453c13f1ee4c7bcfb6d40ea@cfpb.gov"},"message":"Merge pull request #5408 from chosak/homepage-infounit-ordering\n\nFix new home page info unit ordering","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/288199fca18c34ecb27cc1f7fc9fc84c32ee19d4"},{"sha":"448c4813504d03bce648a1dcddd9fb55bbe4e24e","author":{"name":"Scott Cranfill","email":"4cd1de6d933f21024dec82a28b29de1b8621136e@cfpb.gov"},"message":"Carousel: Update layout code for cross-browser compatibility","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/448c4813504d03bce648a1dcddd9fb55bbe4e24e"},{"sha":"86d0bc17ecf2ccc011e6abd829e0d34a3120d332","author":{"name":"Scott Cranfill","email":"4cd1de6d933f21024dec82a28b29de1b8621136e@cfpb.gov"},"message":"Carousel: Fix overdflowing text on no-js view on narrow screens\n\nCo-Authored-By: null <virginiacc@users.noreply.github.com>","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/86d0bc17ecf2ccc011e6abd829e0d34a3120d332"},{"sha":"d194897199f3e6221d50cbf1773dc3506685ab85","author":{"name":"Shia LaBot","email":"d16c4ab686498ad36d5e7446f3781c9f5833bed7@users.noreply.github.com"},"message":"Merge pull request #5405 from cfpb/carousel-layout\n\nCarousel: Update layout code for cross-browser compatibility","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/d194897199f3e6221d50cbf1773dc3506685ab85"},{"sha":"bf5dab1f2031a045bbb2eaf7e2353a48c0e9f7a5","author":{"name":"Scott Cranfill","email":"4cd1de6d933f21024dec82a28b29de1b8621136e@cfpb.gov"},"message":"New Homepage: Fix link font weight in carousel and cards (#5409)\n\nThey should be medium, not regular.\r\n\r\nAddresses https://GHE/CFPB/el-camino/issues/60#issuecomment-232097","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/bf5dab1f2031a045bbb2eaf7e2353a48c0e9f7a5"},{"sha":"060f5196070164d700fcb4f55486a6ca01793c3d","author":{"name":"virginiacc","email":"44b6969d4b7ba6f4589dd981daaa9f42c42147ee@gmail.com"},"message":"Add a hidden H1 to home page. (#5411)","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/060f5196070164d700fcb4f55486a6ca01793c3d"},{"sha":"bea477ffc4bac6f885f2e95825a804509d711290","author":{"name":"Scott Cranfill","email":"4cd1de6d933f21024dec82a28b29de1b8621136e@cfpb.gov"},"message":"New Homepage: Fix card layout in IE 8 & 9 (#5412)\n\nThese browsers don't support flexbox and needed some fallback styling.\r\n\r\nFixes https://GHE/CFPB/el-camino/issues/50","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/bea477ffc4bac6f885f2e95825a804509d711290"},{"sha":"bef577d8c5e83ee71997113c0e909369bc6e89df","author":{"name":"virginiacc","email":"44b6969d4b7ba6f4589dd981daaa9f42c42147ee@gmail.com"},"message":"Visually hide inactive carousel slides to remove their contents from tab order. (#5413)","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/bef577d8c5e83ee71997113c0e909369bc6e89df"},{"sha":"e200883796b3a5a3752c98dfc85961c5cb5ba531","author":{"name":"Scott Cranfill","email":"4cd1de6d933f21024dec82a28b29de1b8621136e@cfpb.gov"},"message":"Attempt to stop sessionStorage quota exceeded errors (#5414)","distinct":false,"url":"https://api.github.com/repos/cfpb/cfgov-refresh/commits/e200883796b3a5a3752c98dfc85961c5cb5ba531"}]}
{"actor":{"display_login":"higs4281"}}