Skip to content

Conversation

@DexAsHisH
Copy link

Fixes #71

Changes

  • Added conditional rendering to hide "Go-To Interview Problems" section when no problems exist
  • Section now only renders when favoriteProblems.length > 0

Testing

  • ✅ Tested with empty favoriteProblems array (Oyo company) section is hidden.
  • ✅ Linting passed with no new warnings.
  • ✅ Full integration testing completed with local database setup.

Screenshots

Before:
Screenshot 2025-10-07 at 1 03 30 PM
After:
Screenshot 2025-10-07 at 1 04 21 PM

@zaCKoZAck0 zaCKoZAck0 requested a review from Copilot October 7, 2025 10:57
Copy link

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 fixes an issue where an empty "Go-To Interview Problems" section was being displayed when no favorite problems exist for a company. The fix adds conditional rendering to only show this section when there are problems to display.

  • Added conditional rendering using favoriteProblems.length > 0 check
  • Wrapped the entire section div in a conditional JSX expression

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@zaCKoZAck0
Copy link
Collaborator

Looks good to me, will merge this soon!
Thanks @DexAsHisH !

Feel free to explore the platform and star ⭐ this repo, if not already done.
And let me know if you have any feedback or suggestions.

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.

Hide "Goto problems" section if no problem is present.

2 participants