New case: use Stimulus for the "I don't know the court date yet" toggle

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#7,310 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript, jquery, rails
Lĩnh vực
frontend

Hướng nghiên cứu

Bắt đầu từ checkbox và ô nhập ngày ra tòa trong app/views/casa_cases/new.html.erb, sau đó đọc controller reveal hiện có trong app/javascript/controllers/reveal_controller.js để xem nó có phù hợp không. Phiên bản jQuery trong app/javascript/src/new_casa_case.js cho thấy hành vi hiện tại cần sao chép. Hoàn thành khi trường ẩn và hiện khi bật/tắt, hồ sơ được lưu trong cả hai trường hợp, bundle exec rspec spec/system/casa_cases/new_spec.rb chạy đạt, và tệp jQuery cùng dòng require của nó được xóa.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Difficulty: Easy Good First Issue Help Wanted javascript

What

On /casa_cases/new, checking "I don't know the court date yet" hides the court date field. app/javascript/src/new_casa_case.js does this with jQuery. The existing reveal Stimulus controller (app/javascript/controllers/reveal_controller.js) already toggles a target when a checkbox is clicked.

Checklist

  • In app/views/casa_cases/new.html.erb, wire the toggle-court-date-input checkbox and the court date input to reveal (or a tiny new controller if reveal doesn't fit)
  • Delete app/javascript/src/new_casa_case.js and its require in app/javascript/application.js
  • As [email protected], toggle the checkbox on /casa_cases/new and confirm the field hides and shows, and that the case still saves both ways
  • bundle exec rspec spec/system/casa_cases/new_spec.rb passes

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.
Ngôn ngữ chính
Ruby
Star
382
Fork
547
Merge trung bình
2 ngày 19 giờ
Pull request đã merge (30 ngày)
49

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của rubyforgood/casa

Tất cả issue của rubyforgood/casa

Issue tương tự

Thêm issue về Ruby

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.