MerginMaps / MerginMaps/python-api-client
Check project ID when doing sync
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 23
- Fork
- 12
- Merge trung bình
- 2 ngày 13 giờ
- Pull request đã merge (30 ngày)
- 7
Mô tả
When doing project pull/push, we should check project info based on project's ID instead of project name, to make sure that we're do the right thing even if something happened to the project:
- project got renamed (or project was transferred to other workspace and we still have access to it) - we sync as usual, using the new project name when doing upload/download API calls
- project was deleted - if project info returns 404, we know it got deleted
- project access permission was removed from our user - if project info returns 403, we know there's permission problem
Note: some projects created/updated with an old version of MM plugin will not have project ID stored in the metadata - it would be good if we could handle those correctly as well (just hope that the project was not renamed / transferred).
Once #190 is implemented, it should be possible to add auto tests that will exercise that check.
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 theo dõi luồng pull/push của project và cách metadata của project hiện đang lưu tên project. Xem lại các lệnh gọi API project-info và issue #190, sau đó thêm các kiểm tra cho việc tra cứu dựa trên ID, xử lý việc đổi tên hoặc chuyển giao, xóa khi gặp 404 và lỗi quyền 403, đồng thời giữ nguyên hành vi đối với metadata không có ID. Hoàn thành cần bao gồm các bài kiểm thử tự động được mô tả 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ệ
- python
- Lĩnh vực
- api
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 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
- 42/100