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

Jump to bottom

Chore/move why docs into folders #91

Merged
merged 5 commits into from
Jun 28, 2024
Merged

Chore/move why docs into folders #91

merged 5 commits into from
Jun 28, 2024

Conversation

K-Beicher
Copy link
Contributor

Description

  • This PR moves all the decision posts out into folder.
  • These changes are done because we would like to have the keywords work on the website, and this seems to be the easiest way to get that.

Related Issues

Closes #83

Reviewer Focus

This PR only needs a quick review.

Just to check that all files have been renamed correctly, and the folders have the correct names.

Checklist

For general documentation:

  • Spell-check
    • US
    • UK
  • Did the page(s) preview correctly on your machine without breaking
  • New category words (keywords) (if any) added to the code snippet file

@K-Beicher K-Beicher self-assigned this Jun 28, 2024
@K-Beicher K-Beicher requested a review from a team as a code owner June 28, 2024 11:32
signekb
signekb approved these changes Jun 28, 2024
View reviewed changes
Copy link
Member

@signekb signekb left a comment

Choose a reason for hiding this comment

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

Now the categories work! So nice!
Updated the path on the index page so the posts are still shown and updated the link to the principles 👍

@lwjohnst86 lwjohnst86 merged commit e2a5c57 into main Jun 28, 2024
@lwjohnst86 lwjohnst86 deleted the docs/move-why-docs-into-folders branch June 28, 2024 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lwjohnst86 lwjohnst86 lwjohnst86 approved these changes

@signekb signekb signekb approved these changes

Assignees

@K-Beicher K-Beicher

Labels
None yet
Projects
Team project planning
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug]: Categories not rendered as links
3 participants
@K-Beicher @lwjohnst86 @signekb