Skip to content

Conversation

@sumitshinde-84
Copy link
Collaborator

@sumitshinde-84 sumitshinde-84 commented Jul 16, 2025

Description

This PR adds an author bio card at the end of articles to strengthen E-E-A-T (Expertise, Experience, Authoritativeness, Trustworthiness) signals for SEO. The bio card highlights the author’s credentials, expertise, and social links helping build credibility with both readers and search engines. AI were used to assist with all changes in this PR

The implementation was inspired by the design used in this article:
Industrial IoT Data Streaming & Continuous Intelligence through AI/ML – HiveMQ
Additionally:

  • The byline (author name and publish date) has been moved to the top of the post.
  • Author information has been removed from the right sidebar to reduce visual clutter.
  • A sample bio was added to sumit-shinde.json to demonstrate how the bio card works.
Screen.Recording.2025-07-16.at.12.18.03.PM.mov

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated

@sumitshinde-84 sumitshinde-84 requested a review from Yndira-E July 16, 2025 08:30
@sumitshinde-84
Copy link
Collaborator Author

Hi @Yndira-E, could you please review this PR when you have a moment?

@Yndira-E
Copy link
Contributor

Hey @sumitshinde-84 a couple of notes:

  • It would make more sense to say the date first and then the author; it would read more naturally.
  • Please move the sticky style to the related article section so the form doesn't get stuck below the fold

I'm not crazy about the location of the date and author, but we can iterate on that later.

Also, as a follow-up, we could move the related articles to the bottom, we could try something similar to:
Screenshot 2025-07-17 at 12 53 08

But with less text!

@sumitshinde-84
Copy link
Collaborator Author

@Yndira-E Done I have applied those 2 changes you suggested

@sumitshinde-84
Copy link
Collaborator Author

@Yndira-E For related articles I will create another PR

{% if people[author].github %}
<a href="https://github.com/{{ people[author].github }}" target="_blank" class="text-gray-600 hover:text-indigo-600 transition-colors" title="GitHub">
<svg class="w-5 h-5" fill="currentColor" viewBox="0 0 20 20">
<path fill-rule="evenodd" d="M10 0C4.477 0 0 4.484 0 10.017c0 4.425 2.865 8.18 6.839 9.504.5.092.682-.217.682-.483 0-.237-.008-.868-.013-1.703-2.782.605-3.369-1.343-3.369-1.343-.454-1.158-1.11-1.466-1.11-1.466-.908-.62.069-.608.069-.608 1.003.07 1.531 1.032 1.531 1.032.892 1.53 2.341 1.088 2.91.832.092-.647.35-1.088.636-1.338-2.22-.253-4.555-1.113-4.555-4.951 0-1.093.39-1.988 1.029-2.688-.103-.253-.446-1.272.098-2.65 0 0 .84-.27 2.75 1.026A9.564 9.564 0 0110 4.844c.85.004 1.705.115 2.504.337 1.909-1.296 2.747-1.027 2.747-1.027.546 1.379.203 2.398.1 2.651.64.7 1.028 1.595 1.028 2.688 0 3.848-2.339 4.695-4.566 4.942.359.31.678.921.678 1.856 0 1.338-.012 2.419-.012 2.747 0 .268.18.58.688.482A10.019 10.019 0 0020 10.017C20 4.484 15.522 0 10 0z" clip-rule="evenodd"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not use {% include "components/github.njk" %} like we do in the team page?
It would be cleaner.

And same for all icons

@sumitshinde-84 sumitshinde-84 removed the request for review from ZJvandeWeg July 17, 2025 15:35
@sumitshinde-84 sumitshinde-84 requested a review from Yndira-E July 18, 2025 14:04
@sumitshinde-84
Copy link
Collaborator Author

@Yndira-E I’ve just made an update could you please take a look?

@Yndira-E
Copy link
Contributor

Looks good! But I don't see any updates to the base.njk file. Weren't you going to add the published and updated dates as meta tags?

@sumitshinde-84
Copy link
Collaborator Author

@Yndira-E Done, added meta to base.njk

@ZJvandeWeg
Copy link
Member

@sumitshinde-84 At what stage will you just fork the upstream library we depend on JSON-LD?

@sumitshinde-84
Copy link
Collaborator Author

@ZJvandeWeg Yes, I’ll do that. I’ll prepare to fork the upstream JSON-LD library and ensure it aligns with what we need and will create PR to make changes to our site

@sumitshinde-84
Copy link
Collaborator Author

Hey @sumitshinde-84 a couple of notes:

  • It would make more sense to say the date first and then the author; it would read more naturally.
  • Please move the sticky style to the related article section so the form doesn't get stuck below the fold

I'm not crazy about the location of the date and author, but we can iterate on that later.

Also, as a follow-up, we could move the related articles to the bottom, we could try something similar to: Screenshot 2025-07-17 at 12 53 08

But with less text!

for the related article here is the PR: #3516

@sumitshinde-84 sumitshinde-84 merged commit 8102359 into main Jul 18, 2025
5 checks passed
@sumitshinde-84 sumitshinde-84 deleted the author-bio-only branch July 18, 2025 15:56
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.

4 participants