Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: render Markdown in rich parameter descriptions #6098

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 10 commits into from
Feb 8, 2023
Merged

feat: render Markdown in rich parameter descriptions #6098

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ea09f5e
WIP
mtojek Feb 7, 2023
a3d6df1
Render description_markdown with glow
mtojek Feb 7, 2023
d6477c1
API tests
mtojek Feb 8, 2023
dfffe4d
Description plaintext
mtojek Feb 8, 2023
ccffe0f
Site markdown
mtojek Feb 8, 2023
b804fe6
Description can be optional
mtojek Feb 8, 2023
63b40c1
Merge branch 'main' into 5931-description-markdown-2
mtojek Feb 8, 2023
b93ea13
Fix
mtojek Feb 8, 2023
4dda465
Fix: mock
mtojek Feb 8, 2023
b0ade51
Leave newlines and links
mtojek Feb 8, 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
Next Next commit
WIP
  • Loading branch information
@mtojek
mtojek committed Feb 7, 2023
commit ea09f5e234316344b63eee66beed91aa0e121099
4 changes: 2 additions & 2 deletions cli/cliui/parameter.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,8 @@ func ParameterSchema(cmd *cobra.Command, parameterSchema codersdk.ParameterSchem

func RichParameter(cmd *cobra.Command, templateVersionParameter codersdk.TemplateVersionParameter) (string, error) {
_, _ = fmt.Fprintln(cmd.OutOrStdout(), Styles.Bold.Render(templateVersionParameter.Name))
if templateVersionParameter.Description != "" {
_, _ = fmt.Fprintln(cmd.OutOrStdout(), " "+strings.TrimSpace(strings.Join(strings.Split(templateVersionParameter.Description, "\n"), "\n "))+"\n")
if templateVersionParameter.DescriptionPlaintext != "" {
_, _ = fmt.Fprintln(cmd.OutOrStdout(), " "+strings.TrimSpace(strings.Join(strings.Split(templateVersionParameter.DescriptionPlaintext, "\n"), "\n "))+"\n")
}

var err error
Expand Down
7 changes: 7 additions & 0 deletions coderd/parameter/plaintext.go
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
package parameter

// Plaintext function converts the description with optional Markdown tags
// to the plaintext form.
func Plaintext(markdown string) string {
return ""
}
22 changes: 22 additions & 0 deletions coderd/parameter/plaintext_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
package parameter_test

import (
"testing"

"github.com/stretchr/testify/require"

"github.com/coder/coder/coderd/parameter"
)

func TestPlaintext(t *testing.T) {
t.Parallel()
t.Run("Simple", func(t *testing.T) {
t.Parallel()

mdDescription := `# Provide the machine image
See the [registry](https://container.registry.blah/namespace) for options.
`
stripped := parameter.Plaintext(mdDescription)
require.Equal(t, "AAA", stripped)
})
}
26 changes: 14 additions & 12 deletions coderd/templateversions.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1458,19 +1458,21 @@ func convertTemplateVersionParameter(param database.TemplateVersionParameter) (c
Icon: option.Icon,
})
}

return codersdk.TemplateVersionParameter{
Name: param.Name,
Description: param.Description,
Type: param.Type,
Mutable: param.Mutable,
DefaultValue: param.DefaultValue,
Icon: param.Icon,
Options: options,
ValidationRegex: param.ValidationRegex,
ValidationMin: param.ValidationMin,
ValidationMax: param.ValidationMax,
ValidationError: param.ValidationError,
ValidationMonotonic: codersdk.ValidationMonotonicOrder(param.ValidationMonotonic),
Name: param.Name,
Description: param.Description,
DescriptionPlaintext: parameter.Plaintext(param.Description),
Type: param.Type,
Mutable: param.Mutable,
DefaultValue: param.DefaultValue,
Icon: param.Icon,
Options: options,
ValidationRegex: param.ValidationRegex,
ValidationMin: param.ValidationMin,
ValidationMax: param.ValidationMax,
ValidationError: param.ValidationError,
ValidationMonotonic: codersdk.ValidationMonotonicOrder(param.ValidationMonotonic),
}, nil
}

Expand Down
25 changes: 13 additions & 12 deletions codersdk/templateversions.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,18 +33,19 @@ const (

// TemplateVersionParameter represents a parameter for a template version.
type TemplateVersionParameter struct {
Name string `json:"name"`
Description string `json:"description"`
Type string `json:"type" enums:"string,number,bool"`
Mutable bool `json:"mutable"`
DefaultValue string `json:"default_value"`
Icon string `json:"icon"`
Options []TemplateVersionParameterOption `json:"options"`
ValidationError string `json:"validation_error,omitempty"`
ValidationRegex string `json:"validation_regex,omitempty"`
ValidationMin int32 `json:"validation_min,omitempty"`
ValidationMax int32 `json:"validation_max,omitempty"`
ValidationMonotonic ValidationMonotonicOrder `json:"validation_monotonic,omitempty" enums:"increasing,decreasing"`
Name string `json:"name"`
Description string `json:"description"`
DescriptionPlaintext string `json:"description_plaintext,omitempty"`
Type string `json:"type" enums:"string,number,bool"`
Mutable bool `json:"mutable"`
DefaultValue string `json:"default_value"`
Icon string `json:"icon"`
Options []TemplateVersionParameterOption `json:"options"`
ValidationError string `json:"validation_error,omitempty"`
ValidationRegex string `json:"validation_regex,omitempty"`
ValidationMin int32 `json:"validation_min,omitempty"`
ValidationMax int32 `json:"validation_max,omitempty"`
ValidationMonotonic ValidationMonotonicOrder `json:"validation_monotonic,omitempty" enums:"increasing,decreasing"`
}

// TemplateVersionParameterOption represents a selectable option for a template parameter.
Expand Down