Skip to content

Update LatestMergedPRs and ContributionTimeline for consistent empty-…#454

Open
Gaurika-05 wants to merge 1 commit into
PRODHOSH:mainfrom
Gaurika-05:feat/improve-empty-state
Open

Update LatestMergedPRs and ContributionTimeline for consistent empty-…#454
Gaurika-05 wants to merge 1 commit into
PRODHOSH:mainfrom
Gaurika-05:feat/improve-empty-state

Conversation

@Gaurika-05

@Gaurika-05 Gaurika-05 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

…state UI

Summary

This PR updates the LatestMergedPRs and ContributionTimeline component to correctly handle scenarios where no data is available.This update introduces a empty-state message to ensure the user is clearly informed when there is no activity to display.

Related Issue

Closes #317

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor
  • Chore / dependency update

Changes Made

  • Modified LatestMergedPRs.tsx and ContributionTimeline to include an empty-state check before rendering.
  • Added a fallback UI message to display when the merged PRs array is empty or undefined.

AI Usage

  • I did not use AI for any part of the code in this PR
  • I used AI for coding (specify which tool below) and I fully understand every change I made, including which functions I changed, why I changed them, and what side effects they could create

Screenshots (if UI change)

Screenshot (421) Screenshot (422)

Checklist

  • I was assigned to the issue before opening this PR
  • My branch is up to date with main
  • Code works locally and I have tested it
  • No console.log left in src/
  • If schema changed — both schema.sql and a new migration file are included
  • If this is a UI change — I read DESIGN.md and followed the design system (colors, spacing, typography, components)
  • Docs updated if needed
  • PR title follows Conventional Commits format (feat:, fix:, docs:, etc.)
  • This PR description is written in my own words

Summary by CodeRabbit

  • Bug Fixes
    • Added clear empty-state messages when no contribution activity is available.
    • Added a “Latest Merged Pull Requests” section with an informative message when no merged pull requests are found.
    • Improved profile page consistency by displaying helpful content instead of blank areas.

@github-actions

Copy link
Copy Markdown
Contributor

Welcome to OSSfolio, @Gaurika-05! 🎉

Thank you for opening this pull request and contributing to the open-source community! 🚀

To ensure a smooth review process, please make sure you have:

  • Checked that your changes work locally and compile cleanly.
  • Followed the guidelines outlined in CONTRIBUTING.md.
  • Linked your PR to an open issue (e.g. Closes #ISSUE_NUMBER).

We will review your PR as soon as possible. Happy coding! 💻✨

@github-actions github-actions Bot added frontend Related to UI / Next.js UI Visual / design changes labels Jul 14, 2026
@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 57bfc333-3708-4e23-aeef-e2b3fda8ad44

📥 Commits

Reviewing files that changed from the base of the PR and between 469f704 and 9ba3e8d.

📒 Files selected for processing (2)
  • src/components/profile/ContributionTimeline.tsx
  • src/components/profile/LatestMergedPRs.tsx

📝 Walkthrough

Walkthrough

Profile contribution and merged pull request components now render styled empty-state panels instead of disappearing when their data collections are empty.

Changes

Profile empty-state rendering

Layer / File(s) Summary
Empty-state UI
src/components/profile/ContributionTimeline.tsx, src/components/profile/LatestMergedPRs.tsx
ContributionTimeline displays a no-activity panel, while LatestMergedPRs displays a latest-merged-PRs section with a no-results message when data is absent or empty.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related issues

  • Issue 317: Directly covers the empty state added to LatestMergedPRs.
  • Issue 255: Introduced the contribution timeline whose empty state is updated here.

Possibly related PRs

Suggested labels: ELUSOC, completed, NEWBIE

Suggested reviewers: prodhosh

Poem

I’m a bunny with a panel to show,
When activity’s quiet and lists have no glow.
“No contributions yet,” the soft message sings,
“Merged pull requests” await future things.
Hop, hop—empty states now bloom! 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning ContributionTimeline was changed even though linked issue #317 only requested LatestMergedPRs. Move the ContributionTimeline empty-state change to a separate PR or add a linked issue covering that scope.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the empty-state updates to LatestMergedPRs and ContributionTimeline.
Linked Issues check ✅ Passed LatestMergedPRs now checks for empty or missing PRs and renders a friendly fallback, satisfying issue #317.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration.


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.

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

Labels

frontend Related to UI / Next.js UI Visual / design changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[GOOD FIRST ISSUE] Improve empty state for LatestMergedPRs

1 participant