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

Add website reorganization proposal #21

Closed
wants to merge 3 commits into from

Conversation

dfarrell07
Copy link
Member

Propose reorganizing Getting Started and Operations sections into
K8s-inspired Concepts, Tasks, Tutorials, and Reference sections.

Signed-off-by: Daniel Farrell [email protected]

@dfarrell07 dfarrell07 added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 18, 2020
@dfarrell07 dfarrell07 added this to the 0.8.1 milestone Dec 18, 2020
@dfarrell07 dfarrell07 self-assigned this Dec 18, 2020
Propose reorganizing Getting Started and Operations sections into
K8s-inspired Concepts, Tasks, Tutorials, and Reference sections.

Signed-off-by: Daniel Farrell <[email protected]>
@dfarrell07 dfarrell07 removed this from the 0.8.1 milestone Dec 23, 2020
@dfarrell07 dfarrell07 added this to the 0.9-m1 milestone Jan 28, 2021
Base automatically changed from master to devel February 4, 2021 09:43
@mangelajo mangelajo removed this from the 0.9-m1 milestone Feb 24, 2021
@submariner-bot
Copy link
Collaborator

🤖 Created branch: z_pr21/dfarrell07/website_reorg

@dfarrell07
Copy link
Member Author

I'm not sure if we should merge this yet, as I don't think actually implementing these website changes is a priority currently.

Comment on lines +79 to +91
├── development
│ ├── _index.md
│ ├── building-testing.md
│ ├── code-review.md
│ ├── release-process.md
│ ├── security.md
│ ├── shipyard
│ │ ├── _index.md
│ │ ├── first-time.md
│ │ └── advanced.md
│ └── website
│ ├── _index.md
│ └── style_guide.md

Choose a reason for hiding this comment

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

Following up the discussion on submariner-io/submariner-website#708 I think it's better to split this section (or nest it) since there are two types of documentation here:

  1. General guidelines (security, code review, contribution guide (which isn't listed here), website style guide).
  2. Specific operational guides (building and testing, releasing, everything shipyard, website contributions)

We could create either two sections on the top level, or nest on inside the other.

If we're following K8s docs example, the first category would be akin to "Contribute" and for the second one I dont see on a formal website but is akin to https://github.com/kubernetes/community/tree/master/contributors/devel

For example, we could have a structure such as:

Suggested change
├── development
│ ├── _index.md
│ ├── building-testing.md
│ ├── code-review.md
│ ├── release-process.md
│ ├── security.md
│ ├── shipyard
│ │ ├── _index.md
│ │ ├── first-time.md
│ │ └── advanced.md
│ └── website
│ ├── _index.md
│ └── style_guide.md
├── contributing
│ ├── _index.md
│ ├── concepts
│ │ ├── code-review.md
│ │ ├── contributing.md # https://submariner.io/development/contribution-guide/
│ │ └── security.md
│ └── tasks
│ ├── building-testing.md
│ ├── release-process.md
│ ├── shipyard
│ │ ├── _index.md
│ │ ├── first-time.md
│ │ └── advanced.md
│ └── website
│ ├── _index.md
│ └── style_guide.md

@maayanf24
Copy link
Contributor

@dfarrell07 - please open an epic for this proposal

@dfarrell07
Copy link
Member Author

@maayanf24 ACK, will-do. And good catch, I also missed this on the list for 0.13 because there is no paired Epic Issue (because this PR is so old).

@dfarrell07 dfarrell07 linked an issue Apr 12, 2022 that may be closed by this pull request
16 tasks
@mkolesnik mkolesnik mentioned this pull request Apr 14, 2022
16 tasks
@dfarrell07
Copy link
Member Author

Closing this as it's not a priority now or likely to be one soon, we can re-open if it becomes pressing.

@dfarrell07 dfarrell07 closed this Jun 6, 2023
@submariner-bot
Copy link
Collaborator

🤖 Closed branches: [z_pr21/dfarrell07/website_reorg]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed documentation Improvements or additions to documentation enhancement New feature or request priority:medium size:large
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Epic: Reorganize website
6 participants