Skip to content

Conversation

Ajax-online
Copy link
Contributor

Zigbee 3.0 RGB+CCT - 2000-6500k

Zigbee 3.0 RGB+CCT - 2000-6500k
@tpmanley tpmanley requested a review from KKlimczukS August 31, 2020 15:16

// Ajax Online
fingerprint manufacturer: "Ajaxonline", model: "AJ-RGBCCT 5 in 1", deviceJoinName: "Ajax Light", mnmn: "SmartThings", vid: "generic-rgbw-color-bulb-2000K-6500K"
fingerprint profileId: "0104", manufacturer: "Ajax online Ltd", model: "AJ_ZB30_GU10", deviceJoinName: "Ajax Light"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please:

  • correct spacing (4 spaces -> 1 tab)
  • add a Raw Description of this device in a comment on the same line after the finger print
  • since manufacturer and model are provided - profileId is not obligatory

What's the color temperature range of this device ?

@Ajax-online
Copy link
Contributor Author

Ajax-online commented Sep 2, 2020 via email

@KKlimczukS
Copy link
Contributor

KKlimczukS commented Sep 2, 2020

Hello, Konrad Thank you very much for contacting us. Colour temperature is 2000-6500k. Could you please send me the raw information I need to add. If possible can you update for me as I am not very good at this and I always get it wrong. Or please send me the format and I can copy paste if this is ok. Please let me know if this is ok. Regards, Ajay Assani

The fingerpring might look like:
fingerprint manufacturer: "Ajax online Ltd", model: "AJ_ZB30_GU10", deviceJoinName: "Ajax Light", mnmn: "SmartThings", vid: "generic-rgbw-color-bulb-2000K-6500K" //Raw Description: insert here

"Raw Description" can be found in device detail page in https://graph.api.smartthings.com/, but You have to connect Your device with Your SmartThings Hub first.

@Ajax-online
Copy link
Contributor Author

Ajax-online commented Sep 2, 2020 via email

@KKlimczukS
Copy link
Contributor

KKlimczukS commented Sep 2, 2020

Hello Konrad, Thank you very much for you quick reply. Here is the raw description 0B 0104 010D 01 08 0000 0003 0004 0005 0006 0008 0300 1000 00 ( what format shall I add this?) Could you let me know if this is ok

Hello Ajay,
Your new fingerprint should be:
fingerprint manufacturer: "Ajax online Ltd", model: "AJ_ZB30_GU10", deviceJoinName: "Ajax Light", mnmn: "SmartThings", vid: "generic-rgbw-color-bulb-2000K-6500K" // Raw Description: 0B 0104 010D 01 08 0000 0003 0004 0005 0006 0008 0300 1000 00

(Raw Description in a comment is for information purposes only)
Please make sure that fields: "manufacturer" and the "model" match those information that are reported by device (they are also in device detail page).

@Ajax-online
Copy link
Contributor Author

Ajax-online commented Sep 3, 2020 via email

@Ajax-online
Copy link
Contributor Author

Ajax-online commented Sep 3, 2020 via email

@KKlimczukS
Copy link
Contributor

Thank you very much Konrad, Also wanted to ask going forward is there a service you can provide where we can send our products to get the right fingerprint(device handler) checked and update I have changed it now so hopefully it will be Ok

Hello Ajay,

Since You changed the code, I don't see any new changes here in this pull request.

regarding Your question:
@shrakuma Please, could You provide the info about the process ?

@KKlimczukS
Copy link
Contributor

Hi Konrad, Also the CCT only goes to 2020-6329K (and not 2000-6500k) I am I doing something wrong.?

Did You check it with another sample? You mentioned that color temperature range is 2000-6500, so maybe the temperature range is different than declared ?

@Ajax-online
Copy link
Contributor Author

Ajax-online commented Sep 4, 2020 via email

@KKlimczukS
Copy link
Contributor

Hello, Konrad Yes I have checked the other samples all same. So the other code I uploaded is ok? When will this be approved

Hello Ajay,
first we need to have a new commit with Your changes (in this pull request). I don't see any other commits besides the initial one.

@Ajax-online
Copy link
Contributor Author

Ajax-online commented Sep 4, 2020 via email

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