Skip to content

Conversation

@mtojek
Copy link
Member

@mtojek mtojek commented Feb 13, 2025

Fixes: #307

This PR enables basic validation for app display name: coder_app has database a constraint on the column length.

@mtojek mtojek self-assigned this Feb 13, 2025
@mtojek mtojek marked this pull request as ready for review February 13, 2025 11:24
@mtojek mtojek requested review from defelmnq and johnstcn February 13, 2025 11:25
Copy link
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this @mtojek !

@mtojek mtojek merged commit 4b3fc65 into main Feb 13, 2025
5 checks passed
@mtojek mtojek deleted the 307-display-app branch February 13, 2025 11:32
@github-actions github-actions bot locked and limited conversation to collaborators Feb 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: coder_app declaration can attempt to overflow length of column workspace_apps.display_name

2 participants