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

drafting where to work #768

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

georgewallace
Copy link
Contributor

No description provided.

@Copilot Copilot bot review requested due to automatic review settings March 18, 2025 20:57
@georgewallace georgewallace requested a review from a team as a code owner March 18, 2025 20:57
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds a new documentation page outlining guidelines for determining where to make contributions across various Elastic product areas.

  • Introduces an overview of contribution pathways for content based on version (9.0+ vs. pre-9.0)
  • Details product-specific sections for Kibana, Elasticsearch, Cloud, and Machine Learning
  • Provides a mapping table for IA sections to product areas

@georgewallace georgewallace added the documentation Improvements or additions to documentation label Mar 18, 2025
@@ -0,0 +1,99 @@

# **How do I know where to contribute?**
Copy link
Contributor

Choose a reason for hiding this comment

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

How do I know where to contribute?

A very brief drive-by comment that this content seems like it should be integrated with or replace what's in https://docs-v3-preview.elastic.dev/elastic/docs-builder/tree/main/contribute/#contribute


1. **Reference/settings content** lives in the new Markdown files in the code repo as it did pre 9.0. This enables folks to make updates that touch code \+ low-level docs in the same PR.
2. **Narrative/conceptual and overview content** lives in [elastic/docs-content](https://github.com/elastic/docs-content). As an example the [ES|QL overview](https://docs-v3-preview.elastic.dev/elastic/docs-content/tree/main/explore-analyze/query-filter/languages/esql) lives in the **Explore & Analyze** section of the narrative docs. But the reference documentation lives in the reference section of the docs [ES|QL reference](https://docs-v3-preview.elastic.dev/elastic/elasticsearch/tree/main/reference/query-languages/esql)
3. **API reference docs** live in the different OpenAPI spec repositories. This is where you need to update API docs published in the new API docs system
Copy link
Contributor

Choose a reason for hiding this comment

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

API reference docs live in the different OpenAPI spec repositories. This is where you need to update API docs published in the new API docs system

If this content is going to persist as a public-facing document that we point all external and internal contributors to, I'm happy to beef up the "how to contribute to API docs" content here too.


### **Kibana**

#### **What has moved to elastic/docs-content?**
Copy link
Contributor

Choose a reason for hiding this comment

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

What has moved to elastic/docs-content?

IMO this feels like info that's only really relevant for internal contributors and overcomplicates things for external contributors. Is it worth splitting this content into separate sections aimed at external contributors (i.e. as simple as possible, open issue or click edit link) and internal contributors (i.e. where did the files I used to have go? how do I backport? etc)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants