Skip to content

Comments

SEO Optimization (Phase 1)#342

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

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

Conversation

@djhemath
Copy link

@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

Copy link
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
Member

@djhemath can you re-create PR? if needed

@djhemath
Copy link
Author

djhemath commented Feb 8, 2026

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
Member

HarshPatel5940 commented Feb 8, 2026

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
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
Member

mhm 🫰

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

Labels

help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants