mirror of
https://github.com/linuxserver/Heimdall.git
synced 2026-03-02 22:56:59 -05:00
Supported App Sync fails upon reaching Tar1090 #433
Labels
No labels
EnhancedApp
FoundationApp
FoundationApp
FoundationApp
bug
closed-issue-activity
enhancement
enhancement
enhancement
good first issue
help wanted
invalid
no-issue-activity
question
research
research
stale
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Heimdall#433
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jpaolo90 on GitHub (Jun 18, 2021).
The following error is thrown:
@Sahasrahla commented on GitHub (Jun 23, 2021):
I believe this is caused because Heimdall's backend hates numbers in app names. I was able to get App Updates working again by essentially undoing this update:
cd Heimdall-2.2.2/app/SupportedApps/
cp -r ./Tar1090/ ./TarTenNinety/
cd TarTenNinety
mv Tar1090.php TarTenNinety.php
nano TarTenNinety.php
-- Edit the lines 1 and 3 to replace "Tar1090" with "TarTenNinety" and save
nano app.json
-- Edit line 3 to replace "Tar1090" with "TarTenNinety" and save
cd ../../..
php artisan register:app all
I actually ran into another similar error with "n8n" -> "nn", but was able to resolve it in the same manner. Now I can "Update App List" in Heimdall again successfully.
github.com/linuxserver/Heimdall-Apps@0a0ff58378github.com/linuxserver/Heimdall-Apps@f646495717@stevietv commented on GitHub (Jun 24, 2021):
I believe these two lines in the definition of an app are the culprit:
github.com/linuxserver/Heimdall@61a5a1a8b0/app/Application.php (L42)github.com/linuxserver/Heimdall@61a5a1a8b0/app/Application.php (L55)they are cleaning the application name by removing anything that isn't a letter.
I think the regex should be updated to
@KodeStar commented on GitHub (Mar 12, 2022):
@stevietv I've pushed that change, thanks