Skip to content

feat(site): use display name field for tasks - #20918

Merged
ssncferreira merged 2 commits into
mainfrom
ssncferreira/site-tasks-display-name
Nov 25, 2025
Merged

feat(site): use display name field for tasks#20918
ssncferreira merged 2 commits into
mainfrom
ssncferreira/site-tasks-display-name

Conversation

@ssncferreira

@ssncferreira ssncferreira commented Nov 25, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

This PR updates the frontend to use the new display_name field for tasks. In the tasks list view, display_name replaces initial_prompt, and in the tasks sidebar, display_name replaces name. This is a follow-up to #20856, which introduced the display_name field in the backend.

Changes

  • Display task.display_name instead of task.initial_prompt in the tasks table
  • Display task.display_name instead of task.name in the task sidebar view
  • Updated mock data to include display_name for all test tasks
  • Added Storybook story to showcase display name rendering with different lengths (short, max length with ellipsis, and edge cases)

Follow-up: #20856
Closes: #20801

@ssncferreira
ssncferreira force-pushed the ssncferreira/site-tasks-display-name branch 4 times, most recently from f613ff0 to 576fe31 Compare November 25, 2025 15:13
@ssncferreira
ssncferreira force-pushed the ssncferreira/site-tasks-display-name branch from 576fe31 to f449045 Compare November 25, 2025 15:18
@ssncferreira
ssncferreira marked this pull request as ready for review November 25, 2025 15:19
DanielleMaywood
DanielleMaywood approved these changes Nov 25, 2025
View reviewed changes

@DanielleMaywood DanielleMaywood left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving under the assumption we're happy for the task name to remain where I've drawn the arrow

image

mafredri
mafredri approved these changes Nov 25, 2025
View reviewed changes

@mafredri mafredri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 👍🏻

Re: @DanielleMaywood's concern, I think the name is more suitable there than the display name (especially considering space constraints), but it could be a bit confusing. I think we can wait for customer feedback on this, though.

@ssncferreira

ssncferreira commented Nov 25, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Approving under the assumption we're happy for the task name to remain where I've drawn the arrow

image

I also had that question, but I think it makes sense to keep showing the task name because we already show the display name in the sidebar. This way, the user is able to more easily identify which task name is associated with the task display name.

Additionally, since the workspace name matches the task name, showing the task name in the top bar helps users connect the task to its associated workspace when navigating between views.

Started an internal thread on slack regarding this topic: https://codercom.slack.com/archives/C0992H8HGCS/p1764086497375829

@ssncferreira
ssncferreira merged commit 2f399ea into main Nov 25, 2025
26 checks passed
@ssncferreira
ssncferreira deleted the ssncferreira/site-tasks-display-name branch November 25, 2025 16:29
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@mafredri mafredri mafredri approved these changes
@DanielleMaywood DanielleMaywood DanielleMaywood approved these changes

Assignees

@ssncferreira ssncferreira

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add display name field for tasks

3 participants

@ssncferreira @mafredri @DanielleMaywood