labelle-org / labelle

Label printing software
Apache License 2.0
27 stars 4 forks source link

Don't assume the default barcode type when handling empty string #26

Closed maresb closed 4 months ago

maresb commented 4 months ago

Addresses https://github.com/labelle-org/labelle/pull/23#discussion_r1581920270

The idea here is that we should be able to adjust the default barcode without introducing any surprise errors. Since different barcodes have different edge cases, changing the default barcode will require manual testing and adjustment. Therefore, I build in an exception with an explanatory message in order to ensure that the proper adjustments will be made during the development process.