Skip to content
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.

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

docs: Clarify that max duration is required #5105

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Dan-Heath
Copy link
Contributor

@Dan-Heath Dan-Heath commented Sep 12, 2024

There is a discrepency between the UI and the docs over the Max Duration field in the UI. The UI says it is optional, but it appears it is required. This was documented incorrectly in the UI steps for creating an alias.

This PR corrects the incorrect step in the creating an alias procedure, and adds some additional text to the targets domain model and CLI topics to reinforce that the max duration is required and Boundary uses the default value if the user does not specify a value.

View the updates in the preview deployment:

@Dan-Heath Dan-Heath added the backport/website Backport changes to `stable-website` and the latest release-branch label Sep 12, 2024
@Dan-Heath Dan-Heath marked this pull request as ready for review September 12, 2024 15:39
@Dan-Heath Dan-Heath requested a review from a team as a code owner September 12, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/website Backport changes to `stable-website` and the latest release-branch docs/concepts website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant