Skip to content

SEO Optimization (Phase 1)#342

Open
djhemath wants to merge 2 commits into
FOSSUChennai:mainfrom
djhemath:feature/seo-optimization
Open

SEO Optimization (Phase 1)#342
djhemath wants to merge 2 commits into
FOSSUChennai:mainfrom
djhemath:feature/seo-optimization

Conversation

@djhemath
Copy link
Copy Markdown

@djhemath djhemath commented Jul 9, 2025

This PR introduces the first phase of SEO improvements for the application.
It addresses foundational issues that can enhance the site's search engine visibility, with more updates planned in future phases.

Related Issue - #85

Changes in this PR:

  1. Corrected incorrected canonical link references
  2. Added proper Schema.org structured data
  3. Updated URLs to use native anchor tags for better semantic structure

Sarveshmrao
Sarveshmrao approved these changes Aug 25, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@Sarveshmrao Sarveshmrao left a comment

Choose a reason for hiding this comment

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

Seems good.

@Sarveshmrao Sarveshmrao added the help wanted Extra attention is needed label Aug 25, 2025
@HarshPatel5940
Copy link
Copy Markdown
Member

HarshPatel5940 commented Feb 6, 2026

@djhemath can you re-create PR? if needed

@djhemath
Copy link
Copy Markdown
Author

djhemath commented Feb 8, 2026 •
edited
Loading

Hey @HarshPatel5940

I'm a bit surprised by this. This PR has been open for over 6 months with no requested changes, and it was closed without any prior notice.

I know open-source projects typically move slow and I waited for this PR to be seen (without spamming).

Asking to "re-create" it at this point doesn't make any sense to me. If the PR was no longer relevant or needed changes, a discussion earlier or a notification at the very least would have helped.

I'd suggest you to learn more about open-source moderation. Give "The Art of Community" by Jono Bacon a try!

I discourage this kinda moderation and I don't wanna "re-create" or just "create" anything anymore here.

-H.

@nammahari nammahari reopened this Feb 8, 2026
@HarshPatel5940
Copy link
Copy Markdown
Member

HarshPatel5940 commented Feb 8, 2026 •
edited
Loading

hello hemath,
I am sorry i closed the PR earlier without any proper explanation. Earlier that day i was closing up a lot of PR's most of which mentioning why or requesting changes first.

In this PR i completely missed everything cause in my mind i thought it was stale and did not knew its need-fullness and forgot to mention this here at the same time.

I will definitely keep in mind and avoid such moderation in future where we simply say "re-create" or close.
Extremely regretful to my actions earlier, and will explore the reference you mentioned as well.

I will also re-ping @nammahari @JustinBenito @Sarveshmrao who mainly maintain the repository to keep them in loop of my actions.

@djhemath
Copy link
Copy Markdown
Author

djhemath commented Feb 8, 2026

Alright, thanks for the consideration. I just got extra context from @nammahari (thanks for the call, I appreciate that) there has been a lot changes in the app since this PR.

I see too many conflicts with this PR now. I'll spend sometime on this, clear these conflicts out and will re-submit this PR.

Thanks for understanding,
H.

@HarshPatel5940
Copy link
Copy Markdown
Member

HarshPatel5940 commented Feb 8, 2026

mhm 🫰

@nammahari
Copy link
Copy Markdown
Collaborator

nammahari commented Mar 28, 2026

@djhemath are you raising a PR again ?

@mithun-srinivasan
Copy link
Copy Markdown

mithun-srinivasan commented Mar 29, 2026

i'm planning to work on this PR
can anyone tell what i want to do now?

@mithun-srinivasan
Copy link
Copy Markdown

mithun-srinivasan commented Mar 31, 2026

Hi! I'm a beginner and currently exploring open source, so I tried testing this PR locally.

I ran the project and checked the <head> section in the browser. I could see the <title> tag, but I wasn’t able to find other SEO-related tags like:

  • meta description
  • Open Graph tags (og:title, og:description, etc.)

I also noticed that metadata is being generated using generateMetadata(Page.HOME) in layout.tsx, so I might be missing something here.

Just wanted to point this out — are these tags expected to appear, or is there something I should check differently?

Thanks!

@mithun-srinivasan
Copy link
Copy Markdown

mithun-srinivasan commented Mar 31, 2026

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Sarveshmrao Sarveshmrao Sarveshmrao approved these changes

@nammahari nammahari Awaiting requested review from nammahari

@JustinBenito JustinBenito Awaiting requested review from JustinBenito

Assignees

@djhemath djhemath

@nammahari nammahari

Labels

help wanted Extra attention is needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@djhemath @HarshPatel5940 @nammahari @mithun-srinivasan @Sarveshmrao