Skip to content

Issue#505#507

Merged
dricazenck merged 2 commits into
Women-Coding-Community:mainfrom
nycotin:issue#505
Jun 28, 2025
Merged

Issue#505#507
dricazenck merged 2 commits into
Women-Coding-Community:mainfrom
nycotin:issue#505

Conversation

@nycotin

@nycotin nycotin commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

Description

New blog post to be published on the website. A new .html file with the content was added, no banner image was provided in the ticket.
While testing, I found an issue with the rendering of the post template, if no banner image was provided. I therefore copied the same rendering login from the blog template and used it in the post template as well. Now the default image is correctly being rendered (see screenshots below).

Change Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Mentor Update
  • Data Update
  • Documentation
  • Other

Related Issue

Issue#505

Screenshots

Blog page after new post was added:

Screenshot 2025-06-27 183306

Post content after new post was added:

Screenshot 2025-06-27 185656

Issue found, compared to blog logic:

Screenshot 2025-06-27 185517

Issue fixed, blog logic implemented:

Screenshot 2025-06-27 185748

Post content after post template was fixed:
Screenshot 2025-06-27 185853

Pull request checklist

Please check if your PR fulfills the following requirements:

  • I checked and followed the contributor guide
  • I have tested my changes locally.
  • I have added a screenshot from the website after I tested it locally

@nycotin nycotin requested a review from a team as a code owner June 27, 2025 17:32
@dricazenck dricazenck merged commit 2e1ab5f into Women-Coding-Community:main Jun 28, 2025
2 checks passed
@nycotin nycotin mentioned this pull request Jun 28, 2025
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants