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

Fix incorrect check for existing files #1738

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Fix incorrect check for existing files #1738

merged 1 commit into from
Nov 15, 2023

Conversation

frenck
Copy link
Member

@frenck frenck commented Nov 15, 2023

Proposed Changes

Fixes an issue where it checks if a directory exists, while it should check if a file exists.

related to #1736

@frenck frenck added the bugfix Inconsistencies or issues which will cause a problem for users or implementors. label Nov 15, 2023
@frenck frenck merged commit 2098610 into main Nov 15, 2023
22 checks passed
@frenck frenck deleted the frenck-2023-1440 branch November 15, 2023 12:01
@rubendediegomartinez
Copy link

rubendediegomartinez commented Nov 15, 2023

All my scripts were disappeared!!! Help

Edited: sorry, not my scritpts, I wanted to say: my flows

@frenck
Copy link
Member Author

frenck commented Nov 15, 2023

@rubendediegomartinez Seems like you are in the wrong place. Just shouting things like that on a random PR, will not help, especially since there is no information either.

You might as well go outside and scream it there.

Please check the issue tab instead, especially #1736.

../Frenck

@rubendediegomartinez
Copy link

@rubendediegomartinez Seems like you are in the wrong place. Just shouting things like that on a random PR, will not help, especially since there is no information either.

You might as well go outside and scream it there.

Please check the issue tab instead, especially #1736.

../Frenck

Sorry, they were my flows, not my scripts...

I will read your reference.
Thanks

@Navaroff
Copy link

All my scripts were disappeared!!! Help

Edited: sorry, not my scritpts, I wanted to say: my flows

The same happened to me. All flows gone and therefore my HA automations.

@frenck
Copy link
Member Author

frenck commented Nov 15, 2023

@Navaroff If you read above, you would know this comment makes no sense to place here... reading is essential, especially here: #1736

@sphinxlady
Copy link

@Navaroff If you read above, you would know this comment makes no sense to place here... reading is essential, especially here: #1736

It does when you realize the fix doesn't actually fix the problem sadly

@hassio-addons hassio-addons locked as too heated and limited conversation to collaborators Nov 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix Inconsistencies or issues which will cause a problem for users or implementors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants