Skip to content

Bugfix power lora loader load button non functional#611

Open
baslack wants to merge 2 commits intorgthree:mainfrom
baslack:bugfix-power-lora-loader-load-button-non-functional
Open

Bugfix power lora loader load button non functional#611
baslack wants to merge 2 commits intorgthree:mainfrom
baslack:bugfix-power-lora-loader-load-button-non-functional

Conversation

@baslack
Copy link
Copy Markdown

@baslack baslack commented Nov 6, 2025

This seems to work on my system, but my knowledge of TypeScript is minimal, so grain of salt.

You mapping is assuming what you get from loraDetails is clean. It is not. You have nulls in the sequence, and so your access is falling down.
r-vage added a commit to r-vage/rgthree-comfy that referenced this pull request Feb 9, 2026
…n crash

Add null/undefined safety checks when loading loras:
- Fallback to empty array if lorasDetails is null/undefined
- Filter out null entries or entries missing the 'file' property
- Prevents crash when lora list has invalid/incomplete data

Fixes issue rgthree#574 - 'Add Lora' button non-functional
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.

1 participant