Skip to content

Remove tech from requirements#12

Open
melkypie wants to merge 1 commit into
MichalKrasowski:masterfrom
melkypie:fix-requirements
Open

Remove tech from requirements#12
melkypie wants to merge 1 commit into
MichalKrasowski:masterfrom
melkypie:fix-requirements

Conversation

@melkypie

Copy link
Copy Markdown

Closes #11

Seems like the manifest.json is referencing a non existent pypi package.
The package if it had existed is not even imported in any of the component files, so no idea why it was there in first place.
Removing it fixes the error in #11 and allows the integration to load

@mariusnauj

Copy link
Copy Markdown

@melkypie so need only to delete manifest.json, and it's working ?

@melkypie

melkypie commented Oct 15, 2022

Copy link
Copy Markdown
Author

Yup need to remove tech requirement from manifest.json and it loaded fine after.

@mariusnauj

Copy link
Copy Markdown

@melkypie so need not to delet all manifest.json file, but one line in that file?

@melkypie

Copy link
Copy Markdown
Author

Look at the changes I did under File changes tab, and do the same for yourself.

@mariusnauj

Copy link
Copy Markdown

@melkypie thank you. WHat version of your tech controller ?

@maniolox

Copy link
Copy Markdown

Thank you for your help. It works. Thanks again.

@Pxvvxp

Pxvvxp commented Oct 20, 2022

Copy link
Copy Markdown

Thanks it worked for me too.

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.

Error after HA upgrade

4 participants