stufro/chordly

Make bin page searchable

Ouverte

#37 ouverte le 21 juil. 2025

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Ruby (11 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (60 étoiles)
Métriques de merge PR
 (Merge moyen 5h 45m) (1 PR mergée en 30 j)

Description

I'd like to be able to search list of chord sheets which have been deleted.

Requirements

  • The existing search_controller.js is used to implement the search functionality
    • You can see examples of how this is used on the chord sheet and set list show pages.
    • For example in the chords sheets _library.html.slim partial, the search controller is attached to a div and the input element is configured, then in _chord_sheet.html.slim partial on line 1 we define the target and line 9 define the search term.
  • A cypress test is added in spec/cypress/e2e/chord_sheets.cy.js
    • Run rake cy:open to run tests whilst developing
    • You can define a test with it.only("...") to make cypress just run that 1 test on reload

Guide contributeur

Direction de recherche
Examinez le search controller.js existant et son utilisation dans les vues de feuilles d'accords et de listes de morceaux, puis appliquez un modèle similaire à la page bin. Référez vous à library.html.slim et chord sheet.html.slim pour la configuration. Ajoutez un test Cypress similaire à ceux existants dans chord sheets.cy.js.
Stack technique
rubyjavascript
Domaine
full stack
Type d'issue
Fonctionnalité
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitRubyJavaScriptCypress
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70