LordMike / MBW.BlueRiiot2MQTT

Utility to map between Blue Riiots pool API, and Home Assistant MQTT
47 stars 2 forks source link

Add Timezone (TZ) support to the image and enable ENV variable to configure #69

Open craigcurtin-dev opened 2 years ago

craigcurtin-dev commented 2 years ago

Currently looking at the logs to debug information from the console is frustrating at the timezone is GMT.

I have manually added it to my image - but would be nice for the master to reflect it and expose configuration through the ENV variables

Craig

LordMike commented 2 years ago

When you added it - was it just an environment variable?

If so, support is already there, and I can just add docs.


From: craigcurtin-dev @.> Sent: Thursday, May 5, 2022 5:57:54 PM To: LordMike/MBW.BlueRiiot2MQTT @.> Cc: Subscribed @.***> Subject: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Currently looking at the logs to debug information from the console is frustrating at the timezone is GMT.

I have manually added it to my image - but would be nice for the master to reflect it and expose configuration through the ENV variables

Craig

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ3E7R5MHQLXTRS3QC3VIPVYFANCNFSM5VFQZOTA. You are receiving this because you are subscribed to this thread.Message ID: @.***>

craigcurtin-dev commented 2 years ago

Nope had to install tzdata and symlink - went through a few steps until I got it to work

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Michael Bisbjerg @.> Sent: Friday, May 6, 2022 6:40:06 PM To: LordMike/MBW.BlueRiiot2MQTT @.> Cc: Craig Curtin @.>; Author @.> Subject: Re: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

When you added it - was it just an environment variable?

If so, support is already there, and I can just add docs.


From: craigcurtin-dev @.> Sent: Thursday, May 5, 2022 5:57:54 PM To: LordMike/MBW.BlueRiiot2MQTT @.> Cc: Subscribed @.***> Subject: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Currently looking at the logs to debug information from the console is frustrating at the timezone is GMT.

I have manually added it to my image - but would be nice for the master to reflect it and expose configuration through the ENV variables

Craig

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ3E7R5MHQLXTRS3QC3VIPVYFANCNFSM5VFQZOTA. You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69#issuecomment-1119388284, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM6OST3POXKDLT62R5QRIUTVITLGNANCNFSM5VFQZOTA. You are receiving this because you authored the thread.Message ID: @.***>

LordMike commented 2 years ago

Ah – cool. Could you make a PR that I can accept – else I might just get around to it at some point 😊

From: craigcurtin-dev @.> Sent: 6. maj 2022 11:03 To: LordMike/MBW.BlueRiiot2MQTT @.> Cc: Michael Bisbjerg @.>; Comment @.> Subject: Re: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Nope had to install tzdata and symlink - went through a few steps until I got it to work

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Michael Bisbjerg @.<mailto:@.>> Sent: Friday, May 6, 2022 6:40:06 PM To: LordMike/MBW.BlueRiiot2MQTT @.<mailto:@.>> Cc: Craig Curtin @.<mailto:@.>>; Author @.<mailto:@.>> Subject: Re: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

When you added it - was it just an environment variable?

If so, support is already there, and I can just add docs.


From: craigcurtin-dev @.<mailto:@.>> Sent: Thursday, May 5, 2022 5:57:54 PM To: LordMike/MBW.BlueRiiot2MQTT @.<mailto:@.>> Cc: Subscribed @.<mailto:@.>> Subject: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Currently looking at the logs to debug information from the console is frustrating at the timezone is GMT.

I have manually added it to my image - but would be nice for the master to reflect it and expose configuration through the ENV variables

Craig

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ3E7R5MHQLXTRS3QC3VIPVYFANCNFSM5VFQZOTA. You are receiving this because you are subscribed to this thread.Message ID: @.<mailto:@.>>

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69#issuecomment-1119388284, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM6OST3POXKDLT62R5QRIUTVITLGNANCNFSM5VFQZOTA. You are receiving this because you authored the thread.Message ID: @.<mailto:@.>>

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69#issuecomment-1119405987, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ2SBWWI4LIPCWIGLS3VITN3ZANCNFSM5VFQZOTA. You are receiving this because you commented.Message ID: @.**@.>>

craigcurtin-dev commented 2 years ago

Yep will do – will document the steps in it and send it over later today

From: Michael Bisbjerg @.> Sent: Friday, May 6, 2022 9:50 PM To: LordMike/MBW.BlueRiiot2MQTT @.> Cc: Craig Curtin @.>; Author @.> Subject: Re: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Ah – cool. Could you make a PR that I can accept – else I might just get around to it at some point 😊

From: craigcurtin-dev @.<mailto:@.>> Sent: 6. maj 2022 11:03 To: LordMike/MBW.BlueRiiot2MQTT @.<mailto:@.>> Cc: Michael Bisbjerg @.<mailto:@.>>; Comment @.<mailto:@.>> Subject: Re: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Nope had to install tzdata and symlink - went through a few steps until I got it to work

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Michael Bisbjerg @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Sent: Friday, May 6, 2022 6:40:06 PM To: LordMike/MBW.BlueRiiot2MQTT @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Cc: Craig Curtin @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>>; Author @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Subject: Re: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

When you added it - was it just an environment variable?

If so, support is already there, and I can just add docs.


From: craigcurtin-dev @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Sent: Thursday, May 5, 2022 5:57:54 PM To: LordMike/MBW.BlueRiiot2MQTT @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Cc: Subscribed @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Subject: [LordMike/MBW.BlueRiiot2MQTT] Add Timezone (TZ) support to the image and enable ENV variable to configure (Issue #69)

Currently looking at the logs to debug information from the console is frustrating at the timezone is GMT.

I have manually added it to my image - but would be nice for the master to reflect it and expose configuration through the ENV variables

Craig

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ3E7R5MHQLXTRS3QC3VIPVYFANCNFSM5VFQZOTA. You are receiving this because you are subscribed to this thread.Message ID: @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>>

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69#issuecomment-1119388284, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM6OST3POXKDLT62R5QRIUTVITLGNANCNFSM5VFQZOTA. You are receiving this because you authored the thread.Message ID: @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>>

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69#issuecomment-1119405987, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH2YJ2SBWWI4LIPCWIGLS3VITN3ZANCNFSM5VFQZOTA. You are receiving this because you commented.Message ID: @.**@.mailto:***@***.******@***.***>>

— Reply to this email directly, view it on GitHubhttps://github.com/LordMike/MBW.BlueRiiot2MQTT/issues/69#issuecomment-1119533699, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM6OST3UVH7MI5SBFPXPODLVIUBORANCNFSM5VFQZOTA. You are receiving this because you authored the thread.Message ID: @.**@.>>

Joe00 commented 2 months ago

any news on this?