MatDialog `open` method `R` generic is too loose
Đang mở
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
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