Check project ID when doing sync
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 42/100
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.
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.
- 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
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.
Issue khác của MerginMaps/python-api-client
-
Vacuuming Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
MerginMaps/python-api-client#320 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
MerginMaps/python-api-client#270 ·
-
Migration tool between servers Đang mởenhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
MerginMaps/python-api-client#257 · 3 bình luận ·
-
enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
MerginMaps/python-api-client#254 · 1 bình luận ·
-
Impose geodiff push limits Đang mởenhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
MerginMaps/python-api-client#245 ·
Tất cả issue của MerginMaps/python-api-client
Issue tương tự
-
Add: hunch Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
DiamondLightSource/dodal#2211 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
openml/openml-python#1749 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
sipyourdrink-ltd/bernstein#6191 ·