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.

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

add Connections crosslinks to Existing app tutorial #1103

Merged
merged 3 commits into from
Apr 24, 2024
Merged

add Connections crosslinks to Existing app tutorial #1103

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
21fa9a3
add Connections crosslinks to Existing app tutorial
willtsai Apr 5, 2024
96164ff
Merge branch 'v0.32' into crosslinks-existing-app-tutorial
willtsai Apr 22, 2024
694db2d
add more crosslinks
willtsai Apr 23, 2024
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
1 change: 1 addition & 0 deletions docs/content/tutorials/tutorial-add-radius/index.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,4 +321,5 @@ This output shows that Radius has detected the three container resources in the

- To learn more about authoring Radius applications, visit the [Authoring applications guide]({{< ref "guides/author-apps" >}})
- To learn more about deploying applications using Radius, visit the [Deploying applications guide]({{< ref "guides/deploy-apps" >}})
- To learn more about using [Radius Connections]({{< ref "guides/author-apps/containers/overview#connections" >}}) to connect your containers and resources, visit the [Connect to dependencies guide]({{< ref "guides/author-apps/containers/howto-connect-dependencies" >}})
Copy link
Contributor

@Reshrahim Reshrahim Apr 8, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Thanks for updating this. The office hours user did not know how to add connections to existing k8s manifests The connections overview doesn't cover the annotation to use to add connections. I think we should add a simplified version of step 6 from helm tutorial in addition to crosslinking.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the suggestion @Reshrahim - I've updated the cross linking to specifically address the community feedback you've mentioned. However, I think there is a separate issue where the Annotations features aren't documented beyond the tutorials, so I've created this issue to track that work: #1117

Copy link
Contributor

Choose a reason for hiding this comment

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

sounds good!

- To learn more about Radius Recipes, visit the [Recipes guide]({{< ref "guides/recipes" >}})
Loading