angular / angular/components

MatDialog `open` method `R` generic is too loose

Đang mở
#12,898 6 bình luận 3 reaction 0 người được giao Xem trên GitHub
area: material/dialog P4 refactoring
Ngôn ngữ chính
TypeScript
Star
25k
Fork
6.8k
Merge trung bình
1 ngày 8 giờ
Pull request đã merge (30 ngày)
91

Mô tả

Because the `R` generic on `MatDialog.open` isn't constrained by arguments to the function, it's effectively always `any`. We should remove the method generic and change the return generic to `{}`, which will force people to explicitly state which type it is when consuming it.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu bằng cách tìm khai báo của phương thức MatDialog.open và kiểm tra cách generic R của nó được định nghĩa và trả về. Cập nhật kiểu generic như mô tả, sau đó xác minh rằng các bên sử dụng kết quả hộp thoại phải cung cấp rõ ràng kiểu dự định của họ và các kiểm tra TypeScript hiện có vẫn vượt qua.

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

Đánh giá

Công nghệ
angular, typescript
Lĩnh vực
frontend
Loại issue
Lỗi
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

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.