1
0
Fork 0
mirror of https://github.com/maziggy/bambuddy.git synced 2026-05-09 08:25:54 +02:00

[PR #582] [CLOSED] [Feature] Rework Archive duplicates tagging #1080

Closed
opened 2026-05-07 00:15:52 +02:00 by BreizhHardware · 0 comments

📋 Pull Request Information

Original PR: https://github.com/maziggy/bambuddy/pull/582
Author: @Keybored02
Created: 3/3/2026
Status: Closed

Base: 0.2.3b1Head: feature/duplicate_rework


📝 Commits (10+)

  • 1da8712 Use SHA256 for duplicate count, show archive_id on card
  • a024d70 Fix double scrollbars on Archive page
  • 3633076 Show reduced SHA256 to identify print file
  • 92e250a Compact duplicate tags, send to father on click
  • 4a69372 Add Hide Duplicates button
  • 74cf749 Add button localisation
  • d950bc5 Refactor duplicate handling to return name/hash pairs
  • 26d996f Fix title attribute to reference original archive ID in ArchiveCard and ArchiveListRow
  • 537b00f Add localization for "view original print" message
  • c9e87f4 Add duplicate sequence and original archive ID to archive responses

📊 Changes

12 files changed (+162 additions, -26 deletions)

View changed files

📝 backend/app/api/routes/archives.py (+47 -5)
📝 backend/app/schemas/archive.py (+2 -0)
📝 backend/app/services/archive.py (+21 -12)
📝 frontend/src/api/client.ts (+2 -0)
📝 frontend/src/i18n/locales/de.ts (+2 -0)
📝 frontend/src/i18n/locales/en.ts (+2 -0)
📝 frontend/src/i18n/locales/fr.ts (+2 -0)
📝 frontend/src/i18n/locales/it.ts (+2 -0)
📝 frontend/src/i18n/locales/ja.ts (+2 -0)
📝 frontend/src/i18n/locales/pt-BR.ts (+2 -0)
📝 frontend/src/i18n/locales/zh-CN.ts (+2 -0)
📝 frontend/src/pages/ArchivesPage.tsx (+76 -9)

📄 Description

Description

This PR proposes a quick fix to improve duplication handling in the archive page. The idea is to improve usability without fully reworking the archiving behavior. This Pr introduces a different logic to tag duplicates, an UI improvement on the tag itself, a more detailed Archive card, and minor fixes to the Archive page.

Fixes #397 and #512

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test addition or update

Changes Made

  • Changed duplicate tagging: now both the name and SHA256 need to match to tag a print as duplicates. Fixes different print being tagged only on the base of the same name
  • "Duplicate" text removed form the tag, now the tag shows the number of reprints
  • Tag is now a button that sends back to the parent print, with outline highlights
  • A new filter Hide Duplicates has been added to the filter bar, with localisation
  • Archive cards included a reduced SHA256 hash to identify associated files and an archive_id to distinguish similar archive entries
  • Fixed double scrollbar appearing in Archive page

Screenshots

image image

Testing

  • I have tested this on my local machine
  • I have tested with my printer model: H2C

Checklist

  • My code follows the project's coding style
  • I have commented my code where necessary
  • I have updated the documentation (if needed)
  • My changes generate no new warnings
  • I have tested my changes thoroughly

Additional Notes


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/maziggy/bambuddy/pull/582 **Author:** [@Keybored02](https://github.com/Keybored02) **Created:** 3/3/2026 **Status:** ❌ Closed **Base:** `0.2.3b1` ← **Head:** `feature/duplicate_rework` --- ### 📝 Commits (10+) - [`1da8712`](https://github.com/maziggy/bambuddy/commit/1da87125102b0791f6ea1cd6d5167b86c5cb9239) Use SHA256 for duplicate count, show archive_id on card - [`a024d70`](https://github.com/maziggy/bambuddy/commit/a024d70d72b61f1e18680bffbfdd93950e1b5f52) Fix double scrollbars on Archive page - [`3633076`](https://github.com/maziggy/bambuddy/commit/3633076d2a7d85493e680559504143dd814d2168) Show reduced SHA256 to identify print file - [`92e250a`](https://github.com/maziggy/bambuddy/commit/92e250a14fd2a87ca96161410757e8649401738d) Compact duplicate tags, send to father on click - [`4a69372`](https://github.com/maziggy/bambuddy/commit/4a693720b5c204bda6a939018e7984c1598ef331) Add Hide Duplicates button - [`74cf749`](https://github.com/maziggy/bambuddy/commit/74cf749035c7af4396e40f012d6250157d4ad4ed) Add button localisation - [`d950bc5`](https://github.com/maziggy/bambuddy/commit/d950bc535bf61a07df9d5ec5b513530a146a11d7) Refactor duplicate handling to return name/hash pairs - [`26d996f`](https://github.com/maziggy/bambuddy/commit/26d996f88fc6babe4654086f5e817ac9c5c1edd8) Fix title attribute to reference original archive ID in ArchiveCard and ArchiveListRow - [`537b00f`](https://github.com/maziggy/bambuddy/commit/537b00f282088f1ef9356856e4b5bbd9d6926845) Add localization for "view original print" message - [`c9e87f4`](https://github.com/maziggy/bambuddy/commit/c9e87f41b92a08c77d0349669249a764aaa19e3c) Add duplicate sequence and original archive ID to archive responses ### 📊 Changes **12 files changed** (+162 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `backend/app/api/routes/archives.py` (+47 -5) 📝 `backend/app/schemas/archive.py` (+2 -0) 📝 `backend/app/services/archive.py` (+21 -12) 📝 `frontend/src/api/client.ts` (+2 -0) 📝 `frontend/src/i18n/locales/de.ts` (+2 -0) 📝 `frontend/src/i18n/locales/en.ts` (+2 -0) 📝 `frontend/src/i18n/locales/fr.ts` (+2 -0) 📝 `frontend/src/i18n/locales/it.ts` (+2 -0) 📝 `frontend/src/i18n/locales/ja.ts` (+2 -0) 📝 `frontend/src/i18n/locales/pt-BR.ts` (+2 -0) 📝 `frontend/src/i18n/locales/zh-CN.ts` (+2 -0) 📝 `frontend/src/pages/ArchivesPage.tsx` (+76 -9) </details> ### 📄 Description ## Description This PR proposes a quick fix to improve duplication handling in the archive page. The idea is to improve usability without fully reworking the archiving behavior. This Pr introduces a different logic to tag duplicates, an UI improvement on the tag itself, a more detailed Archive card, and minor fixes to the Archive page. ## Related Issue <!-- Link to the issue this PR addresses (if applicable) --> Fixes #397 and #512 ## Type of Change <!-- Mark the relevant option with an "x" --> - [x] Bug fix (non-breaking change that fixes an issue) - [x] New feature (non-breaking change that adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update - [ ] Code refactoring - [ ] Performance improvement - [ ] Test addition or update ## Changes Made <!-- List the specific changes made in this PR --> - Changed duplicate tagging: now both the name and SHA256 need to match to tag a print as duplicates. Fixes different print being tagged only on the base of the same name - "Duplicate" text removed form the tag, now the tag shows the number of reprints - Tag is now a button that sends back to the parent print, with outline highlights - A new filter Hide Duplicates has been added to the filter bar, with localisation - Archive cards included a reduced SHA256 hash to identify associated files and an `archive_id` to distinguish similar archive entries - Fixed double scrollbar appearing in Archive page ## Screenshots <!-- If applicable, add screenshots to demonstrate your changes --> <img width="396" height="531" alt="image" src="https://github.com/user-attachments/assets/aa0fc2b3-973a-414d-a6ba-da63dfa71369" /> <img width="438" height="67" alt="image" src="https://github.com/user-attachments/assets/a2e2408c-3bec-4fa5-9fba-183a0e16dd8c" /> ## Testing <!-- Describe how you tested your changes --> - [x] I have tested this on my local machine - [x] I have tested with my printer model: H2C ## Checklist - [x] My code follows the project's coding style - [x] I have commented my code where necessary - [ ] I have updated the documentation (if needed) - [x] My changes generate no new warnings - [x] I have tested my changes thoroughly ## Additional Notes <!-- Add any additional information that reviewers should know --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
BreizhHardware 2026-05-07 00:15:52 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/bambuddy-maziggy-1#1080
No description provided.