pastelsky/bundlephobia

Getting 404 for empty input

Open

#770 ouverte le 4 avr. 2023

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)JavaScript (8 477 stars) (250 forks)batch import
buggood first issue

Description

Describe the bug It looks like it is expected behaviour but I don't think we should allow an empty input call from the frontend.

To Reproduce Don't type anything in the search box and press enter. It takes you to https://bundlephobia.com/package which gives 404.

Expected behavior I believe we should just do nothing when the input is empty.

Let me know if the suggested fix is good and I can make a relevant PR for the same.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
needs maintainer response
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic javascriptbasic react
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
95
Direction de recherche
Examine the search input component (likely in SearchBar.js) to find the form submission handler. Add a conditional check to prevent navigation or API call when the input is empty. Refer to the expected behavior described in the issue.