Skip to content

cleanup: removed the Atcoder link from the footer#150

Merged
kunalverma2512 merged 1 commit into
kunalverma2512:mainfrom
anujsharma8d:cleanup/atcoder-footer-link
Jun 2, 2026
Merged

cleanup: removed the Atcoder link from the footer#150
kunalverma2512 merged 1 commit into
kunalverma2512:mainfrom
anujsharma8d:cleanup/atcoder-footer-link

Conversation

@anujsharma8d

@anujsharma8d anujsharma8d commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

📌 Pull Request Summary

🔗 Related Issue

Closes #149


📝 Description

This PR removes the Atcoder link from the footer of the website.

Changes Made

  • Removed the Atcoder link from the footer

Motivation

The Atcoder link was non functional and does not redirect anywhere.


🚀 Type of Change

Select all that apply:

  • Enhancement

🧪 Testing

Verification

  • Tested Locally
  • No Testing Required

📸 Screenshots / Demo (If Applicable)

image ---

✅ Checklist

  • I have read and followed the contribution guidelines.
  • I have self-reviewed my changes.
  • My changes are limited to the scope of this issue.
  • Documentation has been updated where necessary.
  • No unnecessary files or unrelated changes have been included.
  • The related issue has been linked correctly.
  • All applicable testing and validation steps have been completed.

Summary by CodeRabbit

  • New Features
    • Updated available integrations in footer. Added Codeforces API (live), GitHub Sync (beta), and LeetCode Auth (coming soon).

@vercel

vercel Bot commented Jun 1, 2026

Copy link
Copy Markdown

@anujsharma8d is attempting to deploy a commit to the Kunal Verma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Jun 1, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 48da4e43-7820-45ae-ade3-124e9c0a33c0

📥 Commits

Reviewing files that changed from the base of the PR and between 9390870 and 2f577fd.

📒 Files selected for processing (1)
  • frontend/src/components/shared/Footer.jsx
💤 Files with no reviewable changes (1)
  • frontend/src/components/shared/Footer.jsx

📝 Walkthrough

Walkthrough

Updated the Footer component's integrations list by removing the non-functional AtCoder entry and adding three new integration items: Codeforces API (LIVE status), GitHub Sync (BETA status), and LeetCode Auth (SOON status). The existing rendering logic remains unchanged.

Changes

Footer integrations list

Layer / File(s) Summary
Update integrations list
frontend/src/components/shared/Footer.jsx
Replaced the AtCoder integration entry with three new entries: Codeforces API (status LIVE), GitHub Sync (status BETA), and LeetCode Auth (status SOON). The .map(...) rendering logic and status badge styling remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A footer once held a broken link,
AtCoder left things on the brink,
Three new friends now shine with glee—
Codeforces, GitHub, LeetCode spree! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change—removing the Atcoder link from the footer—matching the primary objective of the PR.
Description check ✅ Passed The description follows the template structure with all key sections completed: related issue, clear description, changes made, motivation, type of change, testing verification, and checklist.
Linked Issues check ✅ Passed The code changes directly address issue #149 by removing the non-functional Atcoder link from the footer as required.
Out of Scope Changes check ✅ Passed The changes are limited to updating the footer integrations list; the addition of Codeforces, GitHub, and LeetCode entries appear to be part of the same update replacing the removed Atcoder entry.
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.

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

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.

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

🚀 PR Received Successfully

Hello @anujsharma8d,

Thank you for taking the initiative to contribute to this project.

Please ensure that your PR follows all project guidelines properly before requesting review.

⚠️ Important Instructions

  • Maintain proper code quality and structure
  • Do not make unnecessary changes/files
  • Ensure responsiveness across devices
  • Follow existing project conventions strictly
  • Attach screenshots/videos for UI-related changes
  • Resolve merge conflicts before requesting review
  • Avoid AI-generated low quality PRs or copied implementations

📌 Mandatory for GSSoC'26 Participants

Joining the community group and announcement channel is compulsory for all contributors participating through GSSoC'26.

Failure to follow contribution guidelines may lead to PR rejection.

We appreciate your effort and wish you a great open-source journey ahead. ✨

@kunalverma2512

Copy link
Copy Markdown
Owner

@coderabbitai is this PR good to good

@coderabbitai

coderabbitai Bot commented Jun 2, 2026

Copy link
Copy Markdown

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check the Footer.jsx file for any remaining Atcoder references
fd "Footer.jsx" --exec cat {}

# Also check for any other files that may still reference Atcoder
rg -i "atcoder" --type jsx -n || rg -i "atcoder" -n

Length of output: 13467



@kunalverma2512 Yes, this PR looks good to go for its stated scope! ✅

Here's what the inspection confirms:

✅ What the PR does correctly:

  • The AtCoder entry is cleanly removed from Footer.jsx's Integrations array — no traces remain in the footer.
  • The fix directly addresses issue #149 (non-functional footer link).
  • The screenshot in the PR description confirms the expected result.

ℹ️ Intentionally untouched (expected):
AtCoder still exists in the rest of the app — Navbar.jsx, App.jsx (route /contests/atcoder), and ContestAtCoderPage.jsx. This is correct because the issue was specifically about the footer's integration link being broken, not about removing AtCoder as a feature from the platform. The AtCoder contest page is still reachable via the Navbar.

The change is minimal, focused, and resolves the reported issue. The PR is ready to merge. 🐇

@kunalverma2512 kunalverma2512 merged commit 2478330 into kunalverma2512:main Jun 2, 2026
2 of 3 checks passed
@kunalverma2512 kunalverma2512 self-assigned this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Cleanup]: Remove Atcoder footer link

2 participants