Delete dead validated_form.js (the case contact form it targeted was redesigned)
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
- 75/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, jquery
- Área
- frontend
Línea de trabajo
Empieza ejecutando grep -rn component-validated-form app/views app/components para confirmar que nada renderiza la clase. Luego revisa la advertencia de millas conducidas en /case_contacts/new como [email protected] y registra el resultado en la descripción del PR. Elimina app/javascript/src/validated_form.js y app/javascript/tests/validated_form.test.js, y quita su require de app/javascript/application.js. Está terminado cuando npm run test y bundle exec rspec spec/system/case_contacts pasen ambos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What
app/javascript/src/validated_form.js (about 290 lines, jQuery) adds client-side warnings to forms with the class .component-validated-form, for example "you entered miles driven for an in-person contact". No view renders .component-validated-form any more: the redesigned case contact form (app/views/case_contacts/form/details.html.erb + case_contact_form_controller.js) replaced it. So the script exits on its first line.
Checklist
- Confirm
grep -rn component-validated-form app/views app/componentsreturns nothing - Check whether the new case contact form still warns when miles driven is entered for a contact that is not in person. Try it at
/case_contacts/newas[email protected]. Put what you find in the PR description. If the warning is gone, that's fine for this PR; open a follow-up issue describing it - Delete
app/javascript/src/validated_form.jsandapp/javascript/__tests__/validated_form.test.js - Remove
require('./src/validated_form')fromapp/javascript/application.js -
npm run testpasses, and so doesbundle exec rspec spec/system/case_contacts
Before you start
- Comment here to be assigned. Setup is
bin/setup, thenbin/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
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de rubyforgood/casa
-
Remove leftover Cloud66 custom_web process from ProcfilePosiblemente ocupada @ShahadatTest la tomó hoy. AbiertoDifficulty: Easy Good First Issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
rubyforgood/casa#7330 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
dependencies Difficulty: Easy Good First Issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
rubyforgood/casa#7325 ·
Los mantenedores suelen responder en 1 día
-
Difficulty: Easy Good First Issue Help Wanted javascript
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
rubyforgood/casa#7311 ·
Los mantenedores suelen responder en 1 día
-
Difficulty: Easy Good First Issue Help Wanted javascript
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
rubyforgood/casa#7310 ·
Los mantenedores suelen responder en 1 día
-
Difficulty: Easy Good First Issue Help Wanted javascript
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
rubyforgood/casa#7309 ·
Los mantenedores suelen responder en 1 día
Todos los issues de rubyforgood/casa
Issues similares
-
JWT::EncodedToken raises TypeError for a non-object headerPosiblemente ocupada @anakinj la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
jwt/ruby-jwt#780 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
zold-io/zold-ruby-sdk#85 ·
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 62/100