Conversation
chpy04
requested changes
Feb 20, 2026
Contributor
chpy04
left a comment
There was a problem hiding this comment.
Not loading on guest page for me and also no admin tools to change the org image
5351e42 to
0031382
Compare
…ardFeng-Sidebar #3872 richard feng sidebar
chpy04
requested changes
Feb 27, 2026
Contributor
chpy04
left a comment
There was a problem hiding this comment.
In addition to comments, we need to be able to modify the icons / links on the home page such that it is possible to replicate the mock
chpy04
requested changes
Mar 7, 2026
Contributor
chpy04
left a comment
There was a problem hiding this comment.
A couple of changes we need to make from a UI perspective:
- In the admin tools, everything related to the guest view should be on the guest view tab of admin tools. The platform image and text should also be on the guest view. Also can you confirm that the description is being used for something else right now?
- If there is no platform image set the landing page loads forever. We have a logo in the github, so lets default to that if nothing is set
- For the link types, if you want to abstract the two concepts as a boolean in the backend that is fine, but that absolutely need to be two different tables in the frontend. The current link type table should only show ones where the boolean is false, and then the guest tab in admin tools should use the same table but only with the links where it is true. The form should then know which page it is being called from and automatically set the boolean without the user ever knowing they are the same thing
- On mobile, the sidebar should open across the whole page not just half, but we can save this for later
- On the guest landing page, the component and button for people joining is too big. Also we can't call it "learn more" bc it makes it way too easy to click it without reading the text above it. UI/UX 101: users don't read.
- The projects aren't centered on mobile view:
…m button, move things to guest view
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Updated the guest home page. Removed references to applyInterestImageId and exploreAsGuestImageId. Added upload platform logo image into admin tools under recruitment tab.
Screenshots
home page:


admin tools:

postman tests:



Checklist
It can be helpful to check the
ChecksandFiles changedtabs.Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.
yarn.lockchanges (unless dependencies have changed)Closes #3873