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

Rendering EOY banner on donate pages #13307

Merged
merged 3 commits into from
Dec 12, 2024
Merged

Conversation

danielfmiranda
Copy link
Collaborator

@danielfmiranda danielfmiranda commented Dec 12, 2024

Description

Related Issue: https://mozilla-hub.atlassian.net/browse/TP1-1477

I found that donate related pages such as donate, /donate/help, and /donate/ways-to-give were not rendering the EOY donate banner. This was because they did not inherit from the class BasePage.

This PR updates the BaseDonationPage class to inherit from BasePage to utilize BasePage's method get_donate_banner(), which adds the EOY donate banner related information to the page context for rendering it on the frontend.

This PR also imports resets.scss in donate-main.scss, which the EOY banner uses to render its featured image in the appropriate dimensions.

Steps to test

  1. Visit https://foundation-s-hotfix-don-38eehi.herokuapp.com/en/donate/.
  2. Confirm the donate banner is rendering.
  3. Visit https://foundation-s-hotfix-don-38eehi.herokuapp.com/en/donate/help/.
  4. Confirm the donate banner is rendering.
  5. Visit https://foundation-s-hotfix-don-38eehi.herokuapp.com/en/donate/ways-to-give/.
  6. Confirm the donate banner is rendering.
  7. If the donate banner is rendering on all 3 pages, testing is complete!

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-hotfix-don-bqcn2a December 12, 2024 02:26 Inactive
@mofodevops mofodevops temporarily deployed to foundation-s-hotfix-don-38eehi December 12, 2024 02:28 Inactive
Copy link
Collaborator

@ramram-mf ramram-mf left a comment

Choose a reason for hiding this comment

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

Thanks Daniel!

Copy link
Collaborator

@dlopezvsr dlopezvsr left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @danielfmiranda!

@danielfmiranda danielfmiranda temporarily deployed to foundation-s-hotfix-don-38eehi December 12, 2024 19:33 Inactive
@danielfmiranda danielfmiranda merged commit 455a6d3 into main Dec 12, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants