Fix issue with skill-based project search and improve skills selection UI#664
Open
Anshkaliya18 wants to merge 4 commits into
Open
Fix issue with skill-based project search and improve skills selection UI#664Anshkaliya18 wants to merge 4 commits into
Anshkaliya18 wants to merge 4 commits into
Conversation
|
@Anshkaliya18 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
This was referenced May 28, 2026
Author
|
Hi maintainers, could you please add the appropriate GSSoC labels to this PR if everything looks good? Thank you. |
komalharshita
requested changes
Jun 5, 2026
Owner
komalharshita
left a comment
There was a problem hiding this comment.
Thanks for the contribution! The multi-select skills UI is a nice improvement and the overall approach looks good. However, a few issues need to be resolved before this can be merged:
- Remove .vscode/settings.json (editor-specific configuration).
- Remove all debugging console.log() statements.
- Revisit the backend ID handling logic to ensure it works safely with all project object types and does not generate invalid IDs.
- Move inline styles into CSS.
- Avoid mixing unrelated section/anchor renaming changes with the skill-search fix.
- Consolidate dropdown visibility management to avoid state inconsistencies.
Please address these points and update the PR for another review.
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.
Fixes #638
Fixes #631
Fixes #640
This PR fixes the issue where users were unable to properly find projects based on selected skills and improves the overall skills selection experience.
Changes Made
Result
Users can now find projects more accurately based on their selected skills with a cleaner and more intuitive UI experience.