Primary heroguts template divided, heroguts homepage created

[dlopezvsr]

Description

This PR divides primary_heroguts.html into two use cases, where the previous "if statements" found together in the current version are separated accordingly. So heroguts_homepage.html was created to isolate HTML related to homepage.html and primary_heroguts.html left for primary_page.html and bannered_campaign_page.html

How to test

• Go to homepage review app and make sure the hero banner still looks good for different viewports.
• Go to some banner campaign page (example) and confirm same of previous point.

For banner page:

For homepage:

Link to sample test page: https://foundation-s-tp1-307-re-08hij9.herokuapp.com/en/
Related PRs/issues: #9605

Checklist

Tests

• Is the code I'm adding covered by tests?

Changes in Models:

• Did I update or add new fake data?
• Did I squash my migration?
• Are my changes backward-compatible. If not, did I schedule a deploy with the rest of the team?

Documentation:

• Is my code documented?
• Did I update the READMEs or wagtail documentation?

Merge Method
💡❗Remember to use squash merge when merging non-feature branches into main

┆Issue is synchronized with this Jira Story

[danielfmiranda]

Hi @dlopezvsr,

Great work on this PR! 🎉 Thanks for taking this on, splitting out the templates will definitely make maintaining the different page types much smoother.

Your description and clear testing steps made reviewing and QA quite easy, so thanks also for that attention to detail.

I’ve walked through the code and followed the test instructions, and everything looks good to me. Approved! 👍

One quick note: I noticed the template names are a little different. One is called heroguts_homepage, while the other is primary_heroguts. It might be clearer if we rename the first one to homepage_heroguts for consistency. No need to hold this PR up, though—feel free to just make that change before merging this PR. Great job again!

[dlopezvsr]

@danielfmiranda thank you so much for the review and the observation. It makes total sense to me and it's definitely clearer that rename. I just updated it in the last commit. 🙌