Skip to content

OUT-3392 | Action items do not reference updated app names#164

Merged
arpandhakal merged 3 commits intomainfrom
OUT-3392
Mar 23, 2026
Merged

OUT-3392 | Action items do not reference updated app names#164
arpandhakal merged 3 commits intomainfrom
OUT-3392

Conversation

@arpandhakal
Copy link
Collaborator

@arpandhakal arpandhakal commented Mar 20, 2026

Summary

  • Fetch app display names from Assembly's listAppInstalls endpoint via a new /api/workspace/app-display-names route
  • Use fetched display names on action cards (label + singular label) with fallback to hardcoded defaults
  • App deployment IDs for Forms, Contracts, and Invoices stored as env vars (FORMS_APP_ID, CONTRACTS_APP_ID, INVOICES_APP_ID)
  • Display names applied across action cards, sidebar settings toggles, and preview mode

Test plan

  • Set FORMS_APP_ID, CONTRACTS_APP_ID, INVOICES_APP_ID env vars
  • Verify action cards show customized app display names from Assembly
  • Verify fallback to default labels when display name is not set
  • Verify sidebar action toggles also show updated names
  • Verify both IU editor and client portal views reflect display names

🤖 Generated with Claude Code

@linear
Copy link

linear bot commented Mar 20, 2026

@vercel
Copy link

vercel bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
client-home-v3 Ready Ready Preview, Comment Mar 23, 2026 10:35am

Request Review

Copy link
Collaborator

@priosshrsth priosshrsth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +89 to +94
const appDeploymentIds = {
forms: env.FORMS_APP_ID,
contracts: env.CONTRACTS_APP_ID,
invoices: env.INVOICES_APP_ID,
tasks: env.TASKS_APP_ID,
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arpandhakal Something to do later, we could store them as config const somwhere else

Copy link
Collaborator Author

@arpandhakal arpandhakal Mar 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the appIds are sensitive data. We should not expose them. Thats why I kept them in .env.

arpandhakal and others added 2 commits March 23, 2026 16:09
…bels

Fetch display names from Assembly's listAppInstalls endpoint and use them
on action cards, with fallback to hardcoded labels. App deployment IDs
are stored as env vars (FORMS_APP_ID, CONTRACTS_APP_ID, INVOICES_APP_ID).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@arpandhakal arpandhakal merged commit 7a2a14b into main Mar 23, 2026
3 of 4 checks passed
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