IQSS / IQSS/dataverse-client-javascript
Refactor ApiRepository for enhanced generality and abstraction
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- TypeScript
- Star
- 20
- Fork
- 11
- Merge trung bình
- 2 ngày 5 giờ
- Pull request đã merge (30 ngày)
- 1
Mô tả
Overview of the Feature Request
The ApiRepository currently incorporates logic that is specific to particular resource types. This approach limits the generic utility of the repository and necessitates direct checks on these types within the repository itself. The proposal is to enhance the generality of ApiRepository by abstracting away resource-specific logic.
What Inspired the Request?
The need for a more generic ApiRepository was identified during a code review. The current implementation requires direct checks on resource types, which can complicate the codebase and reduce the modularity and reusability of the repository.
What Existing Behavior Do You Want Changed?
None, this is just a refactor.
Any Brand New Behavior Do You Want to Add to js-dataverse?
None, this is just a refactor.
Any Open or Closed Issues Related to This Feature Request?
No
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách tìm ApiRepository và xem lại cuộc thảo luận của pull request được liên kết, trong đó đã xác định các kiểm tra dành riêng cho từng tài nguyên. Xác định logic nào gắn với các loại tài nguyên cụ thể và định nghĩa một abstraction để bảo toàn hành vi hiện có; công việc được xem là hoàn tất khi đạt được sự thống nhất về phạm vi refactor và xác minh rằng hành vi hiện tại không thay đổi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- typescript
- Lĩnh vực
- api
- Loại issue
- Tái cấu trúc
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 25/100