commercetools / commercetools/nodejs
Migrate missing features and fixes from old SDK
- Ngôn ngữ chính
- JavaScript
- Star
- 77
- Fork
- 70
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
**Topic for discussion**
It is pity that we hit same [issues](https://github.com/commercetools/nodejs/pull/590) with new SDK which has been addressed in [old SDK](https://github.com/sphereio/sphere-node-sdk/blob/master/src/coffee/sync/product-sync.coffee#L44-L56) already as for existing customers migration might be a costly enterprise as they might hit issues that were not present with old SDK. The reason is obviously that rewrite of new SDK was based on old SDK and as it took around 1 year of development all the fixes/features applied to old SDK during that time were not migrated to the new SDK release.
**Suggestion**
Review all issues/PRs/commits from old SDK implemented after rewrite fork and create a list of TODOs for migration like examples below.
**Missing fixes/features**
- [ ] - Sync actions should first [remove variants](https://github.com/sphereio/sphere-node-sdk/blob/master/src/coffee/sync/product-sync.coffee#L45)
- [ ] - fetch all [convenience function](https://github.com/sphereio/sphere-node-sdk/blob/master/src/coffee/services/base.coffee#L227) for multi-paged results
- [ ] - GraphQL convenience method which allows to do GraphQL requests with single line instead of boiler plate utils in every project
- [ ] - Provide GraphQL process function for paginated results and fetch all if one needs to load all paginated results into memory.
Hướng dẫn đóng góp
Hướng nghiên cứu
Rà soát các tham chiếu SDK cũ trong src/coffee/sync/product-sync.coffee và src/coffee/services/base.coffee, cùng với PR 590 được liên kết. So sánh các issue, PR và commit sau fork với SDK mới, sau đó ghi lại các bản sửa lỗi và tính năng còn thiếu dưới dạng danh sách TODO cho việc di chuyển; công việc được xem là hoàn tất khi có một danh sách đã được review, có thể thực hiện, bao quát các ví dụ trong issue.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- graphql, javascript, node.js
- Lĩnh vực
- api, backend
- Loại issue
- Tính năng
- Độ 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
- 20/100