Skip to content

Generic tape #16857

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

realchrisolin
Copy link

Existing label classes are for specific model printers that are priced outside of the average person using Snipe-IT for personal uses (e.g. bin storage), however there are affordable "generic" Bluetooth label printers available on Amazon and other e-commerce stores. This PR adds the Labels/Tapes/Generic namespace, a GenericTape abstract class, and additional classes that add support for generic 53mm width and 0.59in continuous roll thermal paper printers.

I have another commit in to works to add to this PR that adds another 53mm label that is landscape oriented. The current label (Continuous_53mm_A) is portrait oriented.

@realchrisolin realchrisolin requested a review from snipe as a code owner May 4, 2025 18:57
@realchrisolin realchrisolin changed the base branch from master to develop May 4, 2025 18:58
@snipe
Copy link
Member

snipe commented May 5, 2025

Thanks for this! Can you rebase this PR? I don't think you actually changed 52 files.

@realchrisolin
Copy link
Author

wasn't able to rebase without conflicts, used git rebase develop -Xtheirs to resolve them as gracefully as possible

@snipe
Copy link
Member

snipe commented May 5, 2025

This is still showing 52 files changed, unfortunately - and one of the files you didn't change (the general settings blade) would cause a regression.

@realchrisolin
Copy link
Author

I'm not sure why this is happening. I'm under the impression rebasing off develop should have fast-forwarded this branch (master) and applied my changes on top.

The only other solution I can think of is to check out develop, cherry pick my changes on top, then overwrite this branch with that.

@realchrisolin
Copy link
Author

That seems to have worked, sorry about that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants