Delete attached photos when a feature is deleted
Maintainer thường phản hồi trong vòng 2 ngày
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
- 65/100
Hướng nghiên cứu
No files or tests are named. Start by tracing feature deletion, discard handling, and composition versus association child handling, then verify each acceptance criterion, including nested children, orphaned photos, and the discard confirmation text.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Current situation
When a feature is deleted, its attached photos stay on the device and in the project. This leaves unused files behind, and with the composition relation supported (https://github.com/MerginMaps/mobile/issues/2315) more features will be deleted automatically, so more orphaned photos could pile up.
Proposed solution
When a feature is deleted, delete its attached photos as well. This also applies to child features deleted through a composition relation (for example, if a parent feature is deleted, then its children should be deleted as well)
Acceptance criteria
- Deleting a feature deletes its attached photo(s)
- Deleting a parent with composition children deletes the photo(s) of all deleted child features (including nested ones)
- If child features are created while adding a parent, and the parent is discarded without saving, the photos of those child features are deleted too
- Deleting a parent with association children does not delete the children's photos (and neither the children, check acceptance criteria of (https://github.com/MerginMaps/mobile/issues/2315)
- when discarding a parent feature with composition children, the delete confirmation text mentioned that the attached photos will also be deleted
Related to: https://github.com/MerginMaps/mobile/issues/2315
- Ngôn ngữ chính
- C++
- Star
- 390
- Fork
- 89
- Merge trung bình
- 5 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 13
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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/mobile
-
bluetooth bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
MerginMaps/mobile#4741 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
bug ui/ux
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
MerginMaps/mobile#4669 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
bug Good first issue ui/ux
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
MerginMaps/mobile#4665 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
bug ui/ux
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
MerginMaps/mobile#4647 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
MerginMaps/mobile#4763 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của MerginMaps/mobile
Issue tương tự
-
Missing icon for .cpp2 filesĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 66/100
madskristensen/FileIcons#385 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
microsoft/SysmonCommon#26 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
espressif/esp-matter#1913 ·
-
bug priority: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
chipsalliance/verible#2644 ·