Delete dead sweet-alert-confirm.js (no view uses data-confirm-swal)

Abierto Apto para principiantes
  • ¿Algo no está bien?
#7,306 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
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
85/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript, rails
Área
frontend

Línea de trabajo

Empieza ejecutando el grep de la lista de verificación sobre app/views, app/components y app/helpers para confirmar que nada usa data-confirm-swal. Luego elimina app/javascript/sweet-alert-confirm.js y quita su importación de app/javascript/application.js, dejando el paquete sweetalert2 tal como está. Está terminado cuando las confirmaciones de eliminación y de tipo de contacto siguen apareciendo en el navegador, npm run test pasa y spec/system/volunteers/edit_spec.rb está en verde.

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/sweet-alert-confirm.js hooks Rails UJS so that links with data-confirm-swal show a SweetAlert. No view uses data-confirm-swal any more: confirmations now go through shared/_confirm_button (a native dialog) or plain data-confirm.

Checklist

  • Confirm grep -rn "confirm-swal\|confirm_swal" app/views app/components app/helpers returns nothing
  • Delete app/javascript/sweet-alert-confirm.js and remove its import from app/javascript/application.js
  • Check whether window.Swal is still used (grep -rn "window.Swal\|Swal" app/javascript). Other files import Swal directly, so leave the sweetalert2 package alone in this PR
  • Click a delete button (for example on a contact type) and a deactivate button (on a volunteer's edit page) and confirm both confirmations still appear
  • npm run test passes, and so does bundle exec rspec spec/system/volunteers/edit_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.