Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update search input element id in script.js #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akbaig
Copy link
Contributor

@akbaig akbaig commented Feb 7, 2025

Incorrect search element id was causing the following error:

script.js:34  Uncaught TypeError: Cannot read properties of null (reading 'value')
    at HTMLFormElement.<anonymous> (VM282 script.js:34:58)

@MichaelBelgium
Copy link
Owner

When do you get that error?

It should be q_search tho

<input type="text" name="q" class="form-control" id="q_search" required placeholder="search term" />

@akbaig
Copy link
Contributor Author

akbaig commented Feb 8, 2025

When using dockerfile, the frontend only renders two sections: Convert.php and Search.php

Do you think it fetches the older version of repo through the composer?

image

@MichaelBelgium
Copy link
Owner

Okay, i haven't pushed a new version yet from master. I did it a few seconds ago so docker and composer will pick up the new changes

Can you try it out?

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.

2 participants