Case contacts filter: use the auto-submit Stimulus controller instead of jQuery

Abierto Apto para principiantes
  • ¿Algo no está bien?
#7,309 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript, jquery, rails, ruby
Área
frontend

Línea de trabajo

Empieza con el bloque jQuery turbo:load en app/javascript/src/case_contact.js y con el archivo existente app/javascript/controllers/auto_submit_controller.js. Encuentra un uso que funcione de data-controller="auto-submit" con grep -rn auto-submit app/views y luego aplícalo al formulario de filtros en app/views/case_contacts/index.html.erb. Está terminado cuando cambiar un filtro sigue recargando la lista, npm run test pasa y bundle exec rspec spec/system/case_contacts/index_spec.rb pasa; comprueba si convertDateToSystemTimeZone tiene otros importadores antes de eliminar su prueba.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Difficulty: Easy Good First Issue Help Wanted javascript

What

app/javascript/src/case_contact.js uses jQuery to auto-submit the filter form on /case_contacts (.filter-form .filter-input). There's already a Stimulus controller that does this: app/javascript/controllers/auto_submit_controller.js, used on other filtered tables.

Checklist

  • In app/views/case_contacts/index.html.erb, put data-controller="auto-submit" on the filter form and the matching data-action on the inputs (copy how another page uses it, grep -rn auto-submit app/views)
  • Remove the jQuery turbo:load block from case_contact.js. convertDateToSystemTimeZone is exported and tested in __tests__/case_contact.test.js: keep it if anything still imports it, otherwise delete the file, its test and its require
  • Changing a filter on /case_contacts still reloads the list
  • npm run test passes, and so does bundle exec rspec spec/system/case_contacts/index_spec.rb

Before you start

  • Comment here to be assigned. Setup is bin/setup, then bin/dev. Logins are in the README.
  • JS tests: npm run test. Lint: bin/lint.
  • Part of #7301 (removing jQuery). Keep this PR to just this file.
Lenguaje dominante
Ruby
Estrellas
382
Forks
547
Merge medio
2 d 22 h
PR fusionados (30 d)
46

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de rubyforgood/casa

Todos los issues de rubyforgood/casa

Issues similares

Más issues de Ruby

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.