Skip to content

fix: removed open source text inside TSC#385

Merged
aceppaluni merged 1 commit intohiero-ledger:mainfrom
itsadityaaaaa:remove-open
Apr 13, 2026
Merged

fix: removed open source text inside TSC#385
aceppaluni merged 1 commit intohiero-ledger:mainfrom
itsadityaaaaa:remove-open

Conversation

@itsadityaaaaa
Copy link
Copy Markdown
Contributor

@itsadityaaaaa itsadityaaaaa commented Apr 13, 2026

Description

This PR fixes the inconsistency in the TSC page by removing highlighted open resource text. Now, the box sections behave consistently across the website, where navigation happens by clicking the entire box instead of visible hyperlinks.

Changes Made

  • Removed open resource URLs from the TSC page
  • Modified src/app/tsc by deleting 3 lines
  • Fixed and Ensured consistency with other box sections across the website

Related Issues

Fixes #382

Screenshots (if applicable)

Screenshot from 2026-04-13 19-13-13

Checklist

  • Tests added/updated
  • Documentation updated
  • Linting passes
  • Branch up-to-date with main

Deployment Notes

Summary by CodeRabbit

  • Style
    • Removed the "Open resource" call-to-action text from governance resource cards for a cleaner interface.

Signed-off-by: aditya-verma <vaditya094@gmail.com>
@itsadityaaaaa itsadityaaaaa requested review from a team as code owners April 13, 2026 14:06
Copilot AI review requested due to automatic review settings April 13, 2026 14:06
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 13, 2026

Deploy Preview for hiero-open-source ready!

Name Link
🔨 Latest commit 04507b9
🔍 Latest deploy log https://app.netlify.com/projects/hiero-open-source/deploys/69dcf873f75d2500089104e1
😎 Deploy Preview https://deploy-preview-385--hiero-open-source.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lfdt-bot
Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b29acdb6-f5ea-495d-9d94-a75506a24f75

📥 Commits

Reviewing files that changed from the base of the PR and between 37f8f01 and 04507b9.

📒 Files selected for processing (1)
  • src/app/tsc/page.tsx
💤 Files with no reviewable changes (1)
  • src/app/tsc/page.tsx

📝 Walkthrough

Walkthrough

Removed a decorative "Open resource" text element from governance resource cards on the TSC page to align visual consistency with other box sections throughout the website. Users can still navigate to resources by clicking the card itself.

Changes

Cohort / File(s) Summary
TSC Page UI Cleanup
src/app/tsc/page.tsx
Removed the red underlined "Open resource" call-to-action span from each governance resource card in the TSCSection component.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

status: awaiting-review

Suggested reviewers

  • exploreriii
  • aceppaluni

Poem

🐰 A wee span did hop away,
"Open resource" no more to stay,
The cards now click with grace and cheer,
Consistency blooms bright and clear! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: removed open source text inside TSC' is specific and accurately reflects the main change: removal of visible resource text from the TSC page.
Description check ✅ Passed The description provides clear purpose, lists changes made, references the related issue (#382), includes screenshots, and completes most checklist items.
Linked Issues check ✅ Passed The PR fully addresses issue #382 requirements: removes visible hyperlink text from TSC resource cards, ensuring consistency with other box sections across the site.
Out of Scope Changes check ✅ Passed All changes are scoped to the PR objective; only the TSC page was modified by removing 3 lines of markup related to the 'Open resource' text.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📋 Issue Planner

Built with CodeRabbit's Coding Plans for faster development and fewer bugs.

View plan used: #382

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Timed out fetching pipeline failures after 30000ms


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR resolves the TSC page inconsistency by removing the visible “Open resource” callout text inside the Governance Resources cards, so the cards behave like other “clickable box” sections across the site.

Changes:

  • Removed the “Open resource” text element from each Governance Resources card in the TSC page.
  • Ensured the full card remains the clickable link target without rendering a distinct hyperlink-style label.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@aceppaluni aceppaluni left a comment

Choose a reason for hiding this comment

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

@itsadityaaaaa great work, LGTM!

@aceppaluni aceppaluni merged commit 287d245 into hiero-ledger:main Apr 13, 2026
18 of 19 checks passed
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.

[Beginner]: Remove the open resource text inside the TSC

4 participants