Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(config): added yale fingerprint series #6503

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

catduckgnaf
Copy link

@catduckgnaf catduckgnaf commented Nov 13, 2023

Feat(config): Added YRD410-F-ZW3, YRD420-F-ZW3, YRD430-F-ZW3 & YRD450-F-ZW3

YRD410-F-ZW3 Assure 2 Lock Touch SL Keyed Pushbutton Deadbolt ID 0x4936
YRD420-F-ZW3 Assure 2 Lock Touch SL Keyed Touchscreen Deadbolt ID(s) 0x4929
YRD430-F-ZW3 Assure 2 Lock Touch SL Key Free Pushbutton Deadbolt ID 0x4934
YRD450-F-ZW3 Assure 2 Lock Touch SL Key Free Touchscreen Deadbolt ID(s) 0x4913, 0x4928

@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Nov 13, 2023
@catduckgnaf catduckgnaf changed the title Added Fingerprint Version for Yale Locks Feat(config): Added YRD420-F-ZW3 & YRD450-F-ZW3 Nov 13, 2023
@apella12
Copy link
Contributor

No comment as to content, but I believe your title should be feat(config): added ... if you want to clear the error.

@catduckgnaf catduckgnaf changed the title Feat(config): Added YRD420-F-ZW3 & YRD450-F-ZW3 feat(config): added YRD420-F-ZW3 & YRD450-F-ZW3 Nov 18, 2023
@catduckgnaf
Copy link
Author

No comment as to content, but I believe your title should be feat(config): added ... if you want to clear the error.

thanks, think i got it now

@catduckgnaf
Copy link
Author

Did I get the title right or do I need to do anything else?

@apella12
Copy link
Contributor

I'm not the reviewer/approver. I just thought clearing the error would get more attention from the reviewer. My observation is he/they are pretty busy. Have you added these files to the store/config folder and reinterview ? I think will get them working locally (but not entirely sure).

@catduckgnaf
Copy link
Author

You are right. I know they are. I also have discovered a missing device id. Im gonna close and look. It works locally.

@catduckgnaf catduckgnaf marked this pull request as draft November 28, 2023 16:07
Updated Yale Fingerprint for YRD410-F, 420-F, 430-F, 450-F.
Also updated yale_template.json
@catduckgnaf catduckgnaf changed the title feat(config): added YRD420-F-ZW3 & YRD450-F-ZW3 Feat(config): Added YRD410-F-ZW3, YRD420-F-ZW3, YRD430-F-ZW3 & YRD450-F-ZW3 Nov 28, 2023
@catduckgnaf catduckgnaf changed the title Feat(config): Added YRD410-F-ZW3, YRD420-F-ZW3, YRD430-F-ZW3 & YRD450-F-ZW3 feat(config): Added YRD410-F-ZW3, YRD420-F-ZW3, YRD430-F-ZW3 & YRD450-F-ZW3 Nov 28, 2023
@catduckgnaf catduckgnaf marked this pull request as ready for review November 28, 2023 19:59
@catduckgnaf catduckgnaf changed the title feat(config): Added YRD410-F-ZW3, YRD420-F-ZW3, YRD430-F-ZW3 & YRD450-F-ZW3 feat(config): added yale fingerprint series Nov 28, 2023
@apella12
Copy link
Contributor

Again a tip, but no review. Take a quick look at the details of the failing tests for clues. There is an extra ], on line 11 of one file. There may be more, but that's one. I don't know how you code, but I use VSC and there is some checking built into the node-js repository. (still make mistakes though..)

Copy link
Author

@catduckgnaf catduckgnaf left a comment

Choose a reason for hiding this comment

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

Thank you, looks good. I appreciate you correcting that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙ Configuration issues or updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants