Skip to content

feat: rich parameters: introduce display_name #6919

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Apr 3, 2023
Merged

feat: rich parameters: introduce display_name #6919

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
48e3363
model
mtojek Mar 31, 2023
c4b6b93
DB
mtojek Mar 31, 2023
a11b69e
fix: DisplayName
mtojek Mar 31, 2023
f1aad27
proto
mtojek Mar 31, 2023
e4716cc
Proto
mtojek Mar 31, 2023
89927cb
Update go dep
mtojek Mar 31, 2023
a7bc12c
fixme
mtojek Mar 31, 2023
91d1a1c
Merge branch 'main' into 6852-display-name
mtojek Mar 31, 2023
63296c5
fix format
mtojek Mar 31, 2023
45f6bae
config
mtojek Mar 31, 2023
0f756cf
fmt
mtojek Mar 31, 2023
b14dde1
fix
mtojek Mar 31, 2023
c62f50f
Fix
mtojek Mar 31, 2023
d4411cd
fix
mtojek Mar 31, 2023
1591bbd
chore(UI): redirecting from workspace page if 404 (#6880)
Kira-Pilot Mar 31, 2023
4261fe9
model
mtojek Mar 31, 2023
3aa6ce8
CLI: Display parameter
mtojek Mar 31, 2023
a0554df
fix
mtojek Mar 31, 2023
ad268b8
Merge branch 'main' into 6852-display-name
mtojek Mar 31, 2023
61dab98
update dep
mtojek Mar 31, 2023
a68cc51
Merge branch 'main' into 6852-display-name
mtojek Apr 3, 2023
9fe4100
fix
mtojek Apr 3, 2023
d71c97f
fix
mtojek Apr 3, 2023
3a947d4
fix
mtojek Apr 3, 2023
3ff3b62
UI changes
mtojek Apr 3, 2023
99872e1
fmt
mtojek Apr 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
UI changes
  • Loading branch information
@mtojek
mtojek committed Apr 3, 2023
commit 3ff3b62659059d0b6453cf4807350fe8151b9d1c
12 changes: 12 additions & 0 deletions site/src/components/RichParameterInput/RichParameterInput.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,3 +175,15 @@ DescriptionWithLinks.args = {
options: [],
}),
}

export const BasicWithDisplayName = Template.bind({})
BasicWithDisplayName.args = {
initialValue: "initial-value",
id: "project_name",
parameter: createTemplateVersionParameter({
name: "project_name",
display_name: "Project Name",
description:
"Customize the name of a Google Cloud project that will be created!",
}),
}
5 changes: 3 additions & 2 deletions site/src/components/RichParameterInput/RichParameterInput.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ export interface ParameterLabelProps {
const ParameterLabel: FC<ParameterLabelProps> = ({ id, parameter }) => {
const styles = useStyles()
const hasDescription = parameter.description && parameter.description !== ""
const displayName = parameter.display_name ? parameter.display_name : parameter.name

return (
<label htmlFor={id}>
Expand All @@ -38,13 +39,13 @@ const ParameterLabel: FC<ParameterLabelProps> = ({ id, parameter }) => {

{hasDescription ? (
<Stack spacing={0.5}>
<span className={styles.labelCaption}>{parameter.name}</span>
<span className={styles.labelCaption}>{displayName}</span>
<span className={styles.labelPrimary}>
<MemoizedMarkdown>{parameter.description}</MemoizedMarkdown>
</span>
</Stack>
) : (
<span className={styles.labelPrimary}>{parameter.name}</span>
<span className={styles.labelPrimary}>{displayName}</span>
)}
</Stack>
</label>
Expand Down