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

Add "maintainers" field to package.json #2076

Closed
dkerr64 opened this issue Jun 24, 2024 · 7 comments
Closed

Add "maintainers" field to package.json #2076

dkerr64 opened this issue Jun 24, 2024 · 7 comments
Labels

Comments

@dkerr64
Copy link

dkerr64 commented Jun 24, 2024

Describe The Bug

As #2070 has been closed, opening a new issue to track the need to add "maintainers" field to package.json...

the UI 4.56.2 and below are not detecting that there is a new UI release. To test, I just installed hombridge following instructions here. And it installed UI v4.56.2 and reports that it is up-to-date...
image

image
So your average user has no idea that the Homebridge UI is out-of-date, AND will not get any prompting for ANY other plugin that has had an update published.... unless the plugin owner adds the maintainers list to their package.json.

This UI needs to add it to its own package.json, which will hopefully then cause homebridge to highlight that a new UI is available.

Logs

No response

Config

No response

Homebridge UI Version

4.56.2

Homebridge Version

1.8.3

Node.js Version

20.14

Operating System

Ubuntu / Debian (or a variant)

Environment Info

None

Raspberry Pi Model

None

@mkz212
Copy link
Contributor

mkz212 commented Jun 24, 2024

👍

@mkz212
Copy link
Contributor

mkz212 commented Jun 24, 2024

"maintainers": ["maintainer-name"] to the package.json - that's all !

@NorthernMan54
Copy link
Contributor

Duplicate of #2077

@NorthernMan54 NorthernMan54 marked this as a duplicate of #2077 Jun 24, 2024
@dkerr64
Copy link
Author

dkerr64 commented Jun 24, 2024

Duplicate of #2077

No, not really... this is documenting a specific change that needs to be made to work-around a bug introduced by NPM.

#2077 is a duplicate of #2070 which address a bug in homebridge-config-ui-x (that attempted to dereference a object member without first testing if that object member exists).

Newer issues are normally closed as a dup of an earlier issue, not the other way around.

@dkerr64
Copy link
Author

dkerr64 commented Jun 24, 2024

@NorthernMan54 @mkz212 @donavanbecker Can I respectfully ask that this issue be reopened. It is not fixed and I do not see a fix queued up in the beta-4.56.4 branch... unless you are intending not to fix this, and instead wait on NPM. In which case come out and say that.

I'm not sure why folks are so eager to close valid issues opened to document a necessary change.

@donavanbecker
Copy link
Contributor

@dkerr64 Updating to v4.56.3 does resolve this. So this is resolved.

@dkerr64
Copy link
Author

dkerr64 commented Jun 24, 2024

@dkerr64 Updating to v4.56.3 does resolve this. So this is resolved.

No it doesn't. Maybe I need to restate the problem...

Anyone with 4.56.2 or less will not see any indication that there is a new version of the UI available to install. So yes, the problem is solved in 4.56.3, but the average user is not ever going to install that... they will forever remain on 4.56.2.

The reason is that 4.56.2 throws exception because "maintainers" does not exist, so the UI never shows "update available" so to work around this problem, packages published to npmjs.com must include a maintainers array in the package.json. So that 4.56.2 will not throw an exception when it is missing. This should then restore the notice that an update is available.

Right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants