Skip to content

Show document-list root folder errors - #56

Draft
chee with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-render-sidebar-errors
Draft

Show document-list root folder errors#56
chee with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-render-sidebar-errors

Conversation

Copilot AI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The document-list/sidebar could silently show a loader or fail to render when the account root folder was missing, unfindable, or not actually a folder. This makes those states visible in the sidebar.

  • Root folder validation

    • Detects failed root-folder loads from the Solid resource error state.
    • Treats resolved documents without a docs array as non-folder documents.
    • Avoids rendering DocumentList until the root doc is folder-shaped.
  • Visible error UI

    • Adds a reusable DocumentListError alert.
    • Shows a clear sidebar message for missing rootFolderUrl, failed lookup, or invalid root document.
<Show when={folderLoadError()} keyed fallback={/* document list */}>
  {(error) => (
    <DocumentListError message={error.message} detail={error.detail} />
  )}
</Show>

Copilot AI assigned Copilot and chee Sep 8, 2026
Copilot AI linked an issue Sep 8, 2026 that may be closed by this pull request
Co-authored-by: chee <178266+chee@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix document-list sidebar to show error when rootFolder is invalid Show document-list root folder errors Sep 8, 2026
Copilot AI requested a review from chee September 8, 2026 15:16
@patchcrow

patchcrow commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

✅ Patchwork glance ready

https://patchwork-preview-56--patchwork-base.netlify.app

Patchwork from inkandswitch/patchwork.inkandswitch.com, packages from this PR at d3739c2.

build-info.json

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.

don't silently fail to render sidebar

3 participants