JoarHansson/space-images-app

Change faulty state logic

Ouverte

#4 ouverte le 3 sept. 2024

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)JavaScript (2 forks)auto 404
buggood first issuehacktoberfesthelp wanted

Métriques du dépôt

Stars
 (2 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

  • Faulty logic: setImgLiked(false); in Home.jsx:26 and Home.jsx:68.

    • The logic is faulty because if it assumes the image thats going to show up is not going to be "liked" already, which can indeed be the case.
    • Local storage is already used to save (Home.jsx) and view images (Likes.jsx), and could probably be used here too.
    • A good way to recreate the problem/test a solution is to choose an image by date in the DateInput, like it, choose another image, and then choose it again to see if it's liked.

Guide contributeur

Direction de recherche
Inspectez la logique defectueuse aux lignes 26 et 68 de Home.jsx et utilisez localStorage pour conserver l'etat 'aime' au lieu de toujours le definir a false.
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80