Skip to content

Portal copy and state: no pending state on a new App, wrong descriptions, dead buttons #512

Description

@lonelycode

Small, cheap defects where the product says something that isn't so. Several are one-word changes.

C1 — There is no pending state anywhere in the portal

A portal-created App's credential is minted inactive; an admin must press Approve this App before the key works. The portal says "submitted for approval" — accurate — and then shows the developer an App with a secret they can copy, with nothing anywhere indicating it is not yet live.

They discover it as a 401.

Fix: show the pending state on the App, grey the credential, and say what is being waited on. Two-line change, large support-ticket payoff.

C5 — "No description available" printed under tools that have a description

ui/admin-frontend/src/portal/components/AppDetailView.js:1155

{tool.attributes.short_description || tool.attributes.description || "No description available"}

Renders the fallback in the App page's Tool Access Details even when the Tool carries a description — the field being read is not the one being populated. Same pattern for datasources at :953.

C9 — Typo on a getting-started screen

ui/admin-frontend/src/admin/components/llms/LLMDetails.js:647

"…or have a default URL (for example Anthropic and OopenAI)"

C4a — View in Catalogue goes to the dashboard

On /portal/submissions/{id}, the one button that would close the loop for a contributor navigates to /portal/dashboard. Reproduced twice.

C8 — "PDF export" in Chat is window.print()

With an injected header — browser print-to-PDF, not server-side generation. Reword the control to match.

C10 — Edge "Remove Entry" is cosmetic

A still-running edge re-registers itself. Either implement a real deregister/revoke or say what the button does.


Acceptance

  • Portal App shows a pending state with the credential visibly not-yet-live
  • Tool/datasource descriptions render when present
  • Typo fixed
  • View in Catalogue navigates to the catalogue entry
  • Chat export control names what it does
  • Edge "Remove Entry" either works or explains itself

Note: C3 (Key ID rendering blank) was already fixed — AppDetailView.js:614 now reads key_id. From the UX & UI review of the demo set (section C).

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions