Skip to content

Conversation

jng34
Copy link
Member

@jng34 jng34 commented Oct 8, 2025

Fixes #2008

What changes did you make and why did you make them ?

  • replaced search query that originally used useState hook with useContext hook
  • allows users to retain previous search queries when navigating back and forth

Screenshots of Proposed Changes Of The Website

Visuals of applied changes image image image

@trillium
Copy link
Member

trillium commented Oct 8, 2025

Hey James, thanks for working on another PR! Would you mind double checking the files you staged and comitted? Some Python changes are in your commited code here.

@jng34
Copy link
Member Author

jng34 commented Oct 9, 2025

@trillium which particular file(s) are you referring to? I do not see any changes to any Python code.

@trillium
Copy link
Member

trillium commented Oct 9, 2025

Weird, I could have sworn I saw python changes related to this PR but I don't see them any longer 🤷‍♂️

@JackHaeg
Copy link
Member

Discussed with @trillium on the call - this PR is ready to be reviewed :)

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.

Create user search query hook

3 participants