Skip to content

Add homepage URL to extension.json#99

Merged
niels9001 merged 1 commit into
microsoft:mainfrom
DennieZorg:fix/homepage-missing-crash
Jun 24, 2026
Merged

Add homepage URL to extension.json#99
niels9001 merged 1 commit into
microsoft:mainfrom
DennieZorg:fix/homepage-missing-crash

Conversation

@ScymicX

@ScymicX ScymicX commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Due to a crash when opening the extension from the gallery:

[18:12:45.4391623] [Error] GlobalErrorHandler.cs::HandleException::69 System.ArgumentException: De parameter is onjuist.
at ExtensionGalleryItemPage_obj1_Bindings.Update_ViewModel_Homepage(String, Int32)

Fix: adding missing homepage field to extension.json causes CmdPal to crash
when opening the Game Library Dock entry in the gallery.

Root cause:
ExtensionGalleryItemPage binds Homepage to a UI element that expects a
non-null string. The missing field causes an ArgumentException on page load.

Fix:
Added "homepage": "https://github.com/ScymicX/Steam-Dock-Extension" to
extensions/denniezorg/game-library-dock/extension.json.

Due to a crash when opening the extension from the gallery:

[18:12:45.4391623] [Error] GlobalErrorHandler.cs::HandleException::69 System.ArgumentException: De parameter is onjuist.
   at ExtensionGalleryItemPage_obj1_Bindings.Update_ViewModel_Homepage(String, Int32)

Fix: adding missing `homepage` field to extension.json causes CmdPal to crash 
when opening the Game Library Dock entry in the gallery.

**Root cause:**
`ExtensionGalleryItemPage` binds `Homepage` to a UI element that expects a 
non-null string. The missing field causes an `ArgumentException` on page load.

**Fix:**
Added `"homepage": "https://github.com/ScymicX/Steam-Dock-Extension"` to 
`extensions/denniezorg/game-library-dock/extension.json`.
@niels9001

Copy link
Copy Markdown
Contributor

@ScymicX would you mind filing a bug in the PT repo too? Because it should not crash wheb strings are null.

@niels9001 niels9001 merged commit 3ee3ea5 into microsoft:main Jun 24, 2026
2 checks passed
@ScymicX

ScymicX commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

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.

2 participants