Delete dead validated_form.js (the case contact form it targeted was redesigned)

Aperta Adatta ai principianti
  • Qualcosa non va?
#7,307 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
75/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript, jquery
Ambito
frontend

Direzione di ricerca

Inizia eseguendo grep -rn component-validated-form app/views app/components per confermare che nulla renderizzi la classe. Poi controlla l'avviso sui miglia percorse su /case_contacts/new come [email protected] e registra il risultato nella descrizione della PR. Elimina app/javascript/src/validated_form.js e app/javascript/tests/validated_form.test.js, e rimuovi il loro require da app/javascript/application.js. È completato quando npm run test e bundle exec rspec spec/system/case_contacts passano entrambi.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Difficulty: Intermediate Help Wanted javascript

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/components returns 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/new as [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.js and app/javascript/__tests__/validated_form.test.js
  • Remove require('./src/validated_form') from app/javascript/application.js
  • npm run test passes, and so does bundle exec rspec spec/system/case_contacts

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.
Lingua principale
Ruby
Stelle
382
Fork
547
Merge medio
2g 22h
PR unite (30g)
46

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di rubyforgood/casa

Tutte le issue di rubyforgood/casa

Issue simili

Altre issue su Ruby

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.