Remove quickstarts from docs #683
Conversation
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-683.westus2.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-683.westus2.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://wonderful-plant-020417a1e-683.westus2.azurestaticapps.net |
There was a problem hiding this comment.
some general feedback:
- I feel that the
reference appssection should be nested undertutorialsgiven the similarity in the intent of the content - I'm torn on whether the
languages/bicepsection should be nested undergetting startedas it's a pretty nuanced topic, perhaps it fits better underconcepts?
|
|
||
| - How to use Radius to deploy a Dapr microservices sample application for an online shop | ||
| - How [Dapr and Radius]({{< ref dapr-resources >}}) seamlessly work together | ||
|
|
||
| For more details on the app and access to the source code, visit the `quickstarts/dapr` directory in the [samples repo](https://github.com/project-radius/samples). _For access to the project-radius GitHub org, please complete and submit [this form](https://aka.ms/ProjectRadius/GitHubAccess)._ | ||
| For more details on the app and access to the source code, visit the `tutorials/dapr` directory in the [samples repo](https://github.com/project-radius/samples). _For access to the project-radius GitHub org, please complete and submit [this form](https://aka.ms/ProjectRadius/GitHubAccess)._ |
There was a problem hiding this comment.
Does the samples/quickstarts/... need to be renamed as well?
There was a problem hiding this comment.
Yes the container image names will need to be updated, but that will require a PR into the samples repo and redoing the CI/CD systems. Opened an Issue to track as part of a follow up PR: radius-project/samples#462
There was a problem hiding this comment.
I think this file name is causing issues with how the side bar renders and resulting in the S3 page being blank:
There was a problem hiding this comment.
That's an existing issue that exists already on docs.radapp.dev. I can get that fixed when I go in to remove the reference apps in a follow up PR
Reference apps are also going to be removed in an upcoming PR, with eShop becoming a tutorial. Still working out where to put the others (or remove them). But that will be a follow up PR.
The Bicep page will be converted into a how-to guide (or overview) under deploying apps in a follow up PR. Wanted to focus on just removing quickstarts in this PR. |
❌ Spellcheck FailedThere are spelling errors in your PR. Visit the workflow output to see what words are failing. Adding new wordsIf you are adding a new custom word refer to the docs guide |
|
Closing in favor of #690 |
This PR removes quickstarts, migrating the existing guides into how-to guides and tutorials.
Note that tutorials still need to be documented and contribution guides need to be written, as part of a follow-up PR.