Skip to content

#3872 richard feng sidebar#3955

Open
richard-feng07 wants to merge 3 commits intodevelopfrom
#3872-RichardFeng-Sidebar
Open

#3872 richard feng sidebar#3955
richard-feng07 wants to merge 3 commits intodevelopfrom
#3872-RichardFeng-Sidebar

Conversation

@richard-feng07
Copy link
Contributor

Changes

Added a sidebar to the /home/guest route. Sidebar only shows project and team related pages, along with sponsors and info.

Notes

The design review and sponsors page currently don't lead to anywhere. Also the two images on the /home/guest page aren't working so I just removed them entirely.

Screenshots

Screenshot 2026-02-05 at 4 06 41 PM Screenshot 2026-02-05 at 4 07 05 PM
  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • No yarn.lock changes (unless dependencies have changed)
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #3872

@richard-feng07 richard-feng07 self-assigned this Feb 5, 2026
const { onGuestHomePage } = useHomePageContext();
const [allTeams, setAllTeams] = useState<LinkItem[]>([]);

getAllTeams().then((response) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

We should be displaying all team types, so instead using useAllTeamTypes instead, make sure you are using the one from teams.hooks.ts

setAllTeams(
response.data.map((team) => ({
name: team.teamName,
icon: undefined,
Copy link
Contributor

Choose a reason for hiding this comment

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

Each team type should have an icon associated with it, can you display those here?

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.

[Guest Page Improvements] - Sidebar

2 participants