Skip to content

Fix/community links new tab#2185

Closed
Siddhartha20242 wants to merge 3 commits intojson-schema-org:mainfrom
Siddhartha20242:fix/community-links-new-tab
Closed

Fix/community links new tab#2185
Siddhartha20242 wants to merge 3 commits intojson-schema-org:mainfrom
Siddhartha20242:fix/community-links-new-tab

Conversation

@Siddhartha20242
Copy link
Contributor

What kind of change does this PR introduce?

  • A feature

Issue Number:

Screenshots/videos:

If relevant, did you update the documentation?

No

Summary

This PR updates the Community Section to ensure a better user experience by opening external links (Slack, Discussions, Roadmap, and Blog) in a new browser tab using target='_blank'.

Does this PR introduce a breaking change?
No

Checklist

Please ensure the following tasks are completed before submitting this pull request.

@github-actions
Copy link

github-actions bot commented Jan 31, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview 4e3ca17

@codecov
Copy link

codecov bot commented Jan 31, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.53%. Comparing base (aa0a2ff) to head (4e3ca17).
⚠️ Report is 46 commits behind head on main.

Files with missing lines Patch % Lines
components/DocsHelp.tsx 75.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##              main    #2185      +/-   ##
===========================================
- Coverage   100.00%   99.53%   -0.47%     
===========================================
  Files           30       30              
  Lines          633      651      +18     
  Branches       196      201       +5     
===========================================
+ Hits           633      648      +15     
- Misses           0        3       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@Utkarsh-123github Utkarsh-123github left a comment

Choose a reason for hiding this comment

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

Hi @Siddhartha20242 ,
why have you made changes in DocsHelp.tsx and DocsHelp.cy.tsx ?
I don't think they are relevant with the issue.

@github-project-automation github-project-automation bot moved this from Ready to review to Changes requested in PR - Triage Group Feb 3, 2026
@Siddhartha20242
Copy link
Contributor Author

Hi, @Utkarsh-123github , I am so sorry that i forgot to make a seperate one for this and committed on the same one I will make sure to change this by today midnight:)

@Siddhartha20242
Copy link
Contributor Author

Hi, @Utkarsh-123github , I've cleaned up the PR to only include the community links changes. The codecov check is failing because this change doesn't have test covarage. Could I require the test or if the requirement can be waived for this small UI change? Thanks you so much:)

Copy link
Member

@Utkarsh-123github Utkarsh-123github left a comment

Choose a reason for hiding this comment

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

Image

Hi @Siddhartha20242 , still your PR has irrelevant changes. I am closing this PR and please raise a separate PR on a separate branch that has code changes specifically for this issue.
Thanks!

@github-project-automation github-project-automation bot moved this from Changes requested to Done in PR - Triage Group Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

✨ Enhancement: Community Section links should open in blank tab not in the same tab

2 participants