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(common): Revamp Typical Workflow and simplify First Steps #1974

Merged
merged 16 commits into from
Mar 21, 2024

Conversation

dimodi
Copy link
Contributor

@dimodi dimodi commented Mar 15, 2024

The main changes, which triggered this PR are:

  • Remove defer from the First Steps tutorials.
  • Remove the extra .razor layout from the First Steps tutorials.
  • Move the above to Typical Workflow.

In addition:

  • New CDN article that aims to gather all the information in one place. The article will be updated with the theme CDN after the Themes documentation revamp when we will drop the documentation for the Kendo CDN and replace it with unpkg.
  • Everything else is just polishing and various language and SEO improvements.

@dimodi dimodi added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Mar 15, 2024
@dimodi dimodi requested a review from a team March 15, 2024 13:46
@dimodi dimodi requested a review from a team as a code owner March 15, 2024 13:46
Copy link
Contributor

Hello @dimodi,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

@dimodi dimodi changed the title Getting started defer docs(common): Revamp Typical Workflow and simplify First Steps Mar 15, 2024
Copy link
Contributor

@ikoevska ikoevska left a comment

Choose a reason for hiding this comment

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

Note that I have expressed some concerns over the contents of Typical Workflow and CDN. I believe both pages would benefit from a re-write. The Getting Started section as a whole probably needs some redesign as well. Feel free to log these concerns for later and address them in a separate PR (hence, the approval).

One other thing I would like to recommend going forward is to produce smaller PRs, focused around one particular documentation task. This speeds up the review and the publishing process and, generally, caters to a better structured history within the repo.

getting-started/what-you-need.md Outdated Show resolved Hide resolved
getting-started/what-you-need.md Outdated Show resolved Hide resolved
getting-started/what-you-need.md Outdated Show resolved Hide resolved
getting-started/what-you-need.md Show resolved Hide resolved
getting-started/what-you-need.md Outdated Show resolved Hide resolved
common-features/cdn.md Outdated Show resolved Hide resolved
common-features/cdn.md Show resolved Hide resolved
components/document-processing/overview.md Outdated Show resolved Hide resolved
deployment/overview.md Outdated Show resolved Hide resolved
installation/automated.md Outdated Show resolved Hide resolved
dimodi and others added 8 commits March 21, 2024 17:58
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
Co-authored-by: Iva Stefanova Koevska-Atanasova <[email protected]>
@dimodi dimodi merged commit c029f21 into master Mar 21, 2024
1 check passed
Copy link
Contributor

Here you are the link to your PR to production: 1996

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-to-production Use this label to get a comment to choose whether to merge the PR to production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants