Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

'Printer' 'Port' field recognized as string instead of number #47

Closed
tblossfeld opened this issue Jul 11, 2016 · 3 comments
Closed

'Printer' 'Port' field recognized as string instead of number #47

tblossfeld opened this issue Jul 11, 2016 · 3 comments

Comments

@tblossfeld
Copy link

I am running the updated script, and am running into an error where the script thinks device_port is a string instead of a number. I have set this variable as '6006' in the config file. Error is seen here:

setupmodule

@kdlucas
Copy link
Contributor

kdlucas commented Jul 15, 2016

I think this should be an integer.

Kelly
kdLucas

On Mon, Jul 11, 2016 at 10:07 AM, tblossfeld notifications@github.com
wrote:

I am running the updated script, and am running into an error where the
script thinks device_port is a string instead of a number. I have set this
variable as '6006' in the config file. Error is seen here:

[image: setupmodule]
https://cloud.githubusercontent.com/assets/17885939/16739480/5362740c-4768-11e6-82b2-a2c94f071a68.png


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#47, or mute the
thread
https://github.com/notifications/unsubscribe/ADk_fUxiC5r5nMT48TH8Jn1OVQS4GZqbks5qUng9gaJpZM4JJk6M
.

@kdlucas
Copy link
Contributor

kdlucas commented Jul 15, 2016

I just updated the comments in _config.py to indicate port is an integer
value.

Kelly
kdLucas

On Fri, Jul 15, 2016 at 7:25 AM, K.D. Lucas kdlucas@gmail.com wrote:

I think this should be an integer.

Kelly
kdLucas

On Mon, Jul 11, 2016 at 10:07 AM, tblossfeld notifications@github.com
wrote:

I am running the updated script, and am running into an error where the
script thinks device_port is a string instead of a number. I have set this
variable as '6006' in the config file. Error is seen here:

[image: setupmodule]
https://cloud.githubusercontent.com/assets/17885939/16739480/5362740c-4768-11e6-82b2-a2c94f071a68.png


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#47, or mute the
thread
https://github.com/notifications/unsubscribe/ADk_fUxiC5r5nMT48TH8Jn1OVQS4GZqbks5qUng9gaJpZM4JJk6M
.

@kdlucas
Copy link
Contributor

kdlucas commented Jul 15, 2016

Just set printer port to 6006, not '6006', or let the script automatically update the printer port, as it will try to use MDNS traffic to get the correct port number.

@kdlucas kdlucas closed this as completed Jul 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants