From e75ec6e330823a2e82adecc3dc8b24138fdc842f Mon Sep 17 00:00:00 2001
From: AndBondStyle <andbondstyle@gmail.com>
Date: Mon, 30 Oct 2023 02:24:39 +0300
Subject: [PATCH] Test bluetooth connection, update readme

---
 niimprint/__main__.py |  1 -
 readme.md             | 16 ++++++++++++----
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/niimprint/__main__.py b/niimprint/__main__.py
index 7abaa49..c0eb08d 100644
--- a/niimprint/__main__.py
+++ b/niimprint/__main__.py
@@ -45,7 +45,6 @@ from niimprint import BluetoothTransport, PrinterClient, SerialTransport
 )
 def print_cmd(model, conn, addr, density, image):
     assert model != "d11", "D11 support may be broken (test yourself)"
-    assert conn != "bluetooth", "Bluetooth support may be broken (test yourself)"
 
     if conn == "bluetooth":
         addr = addr.upper()
diff --git a/readme.md b/readme.md
index 744dd0e..c2797b6 100644
--- a/readme.md
+++ b/readme.md
@@ -31,15 +31,23 @@ Options:
 
 ## Examples
 
-**B21, USB connection, 30x15 mm (240x120 px) label.** On linux, you can try to omit `--addr` option and let the script to auto-detect the serial port (it will fail if there're multiple available ports). On windows, serial ports will be named like `COM1`, `COM2` etc. (check in device manager).
+**B21, USB connection, 30x15 mm (240x120 px) label**
 
 ```
-python niimprint --addr /dev/ttyACM0 --image examples/B21_30x15mm_240x120px.png
+python niimprint -a /dev/ttyACM0 -i examples/B21_30x15mm_240x120px.png
 ```
 
-**B21, bluetooth connection:** untested (todo).
+Notes: on linux, you can try to omit `--addr` option and let the script to auto-detect the serial port (it will fail if there're multiple available ports). On windows, serial ports will be named like `COM1`, `COM2` etc. (check in device manager).
 
-**D11:** completly untested, however original fork supports it. If you have D11 at hand and willing to test, please open an issue!
+**B21, bluetooth connection, same label**
+
+```
+python niimprint -c bluetooth -a "E2:E1:08:03:09:87" -i examples/B21_30x15mm_240x120px.png
+```
+
+Notes: it seems like B21 has two bluetooth adresses starting with `C2:E1` and `E2:E1` respectively. It works only if you disconnect from `C2:E1` and connect to `E2:E1`. Also after connecting to `E2:E1` in bluetoothctl I always get `org.bluez.Error.NotAvailable br-connection-profile-unavailable` error, but printing works regardless.
+
+**D11:** completly untested, however original fork supports it. If you have D11 at hand and willing to test, please contact me!
 
 ## Image vs Label size