Skip to content

Replace hero image with carousel slider #529

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

Conversation

TimOsahenru
Copy link
Contributor

Issue Link 🔗:

Issue: #[Issue_Number]
#365

Type of Change

  • Bug fix 🐞
  • New feature/page
  • Documentation update
  • Other

Description 📋

  • What: Provide an overview of the issue this PR addresses. Explain the context and background information.
    Replaced the hero image with carousel slider that displays the post titles, feature images and a summary of the post

  • Why: Describe why the changes are being made. Highlight key updates, new features, or bug fixes.

  • How: Explain how these changes will affect the project or end-users.

Checklist ✅

  • Followed the Code of Conduct and Contribution Guide
  • Ran pre-commit run --all
  • All tests pass locally
  • Added tests (if applicable)
  • Documentation updated (if applicable)

Additional Notes & Screenshots

Add any additional notes or comments that might be helpful for the reviewers.

@kjaymiller kjaymiller added the design_change triggers the generate_screenshot workflow label Oct 14, 2024
@kjaymiller
Copy link
Contributor

It would be good to add some tests to this.

The fact that the page is loading gives me confidence but it would be good to see something.

@TimOsahenru
Copy link
Contributor Author

Hello @kjaymiller, I'm still having trouble getting my tests to pass, as mentioned in a previous pull request. Sometimes all my tests fail, so I was hesitant to include one. Do we have a maintainers' channel where I can share the error message? Maybe someone can help me troubleshoot.

@kjaymiller
Copy link
Contributor

We don't but we can make one definitely.

@TimOsahenru TimOsahenru deleted the replace-hero-image-with-carousel branch October 26, 2024 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design_change triggers the generate_screenshot workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants