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] Creates Developer docs GH page code #5977

Merged
merged 2 commits into from
Mar 11, 2024

Conversation

ashwin-pc
Copy link
Member

@ashwin-pc ashwin-pc commented Feb 28, 2024

Description

Uses Docsify to generate documentation liks that expose Readme's within the OSD repo as HTML.

The GH Pages running on my local fork: https://ashwin-pc.github.io/OpenSearch-Dashboards/docs/index.html#/

Issues Resolved

Screenshot

Screenshot 2024-02-27 at 6 17 02 PM

Testing the changes

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@ashwin-pc ashwin-pc changed the title [Docs] Cretaes Developer docs GH page [Docs] Creates Developer docs GH page Feb 28, 2024
@ashwin-pc ashwin-pc changed the title [Docs] Creates Developer docs GH page [Docs] Creates Developer docs GH page code Feb 28, 2024
Comment on lines +150 to +153
- [Opensearch dashboards.release notes 2.10.0](../release-notes/opensearch-dashboards.release-notes-2.10.0.md)
- [Opensearch dashboards.release notes 2.11.0](../release-notes/opensearch-dashboards.release-notes-2.11.0.md)
- [Opensearch dashboards.release notes 2.11.1](../release-notes/opensearch-dashboards.release-notes-2.11.1.md)
- [Opensearch dashboards.release notes 2.12.0](../release-notes/opensearch-dashboards.release-notes-2.12.0.md)
Copy link
Member

Choose a reason for hiding this comment

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

A little bit weird about the order here.

Copy link
Member Author

Choose a reason for hiding this comment

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

Dont quite know what you mean by the order here. This is simply the folder structure of the OSD repository. What did you expect here?

Copy link
Member

Choose a reason for hiding this comment

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

Should be the orders that 2.1.0 -> 2.9.0 and then 2.10.0 I suppose, but for now the order is 2.1.0 -> 2.10.0 -> 2.12.0 -> 2.2.0, not a semantic version order.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah i see what you mean. Yeah this currently pulls the files as they appear in the filesystem. I could reorder this but I personally liked how they map 1:1 with the file tree :)

Comment on lines +134 to +137
- [Opensearch dashboards.release notes 1.3.10](../release-notes/opensearch-dashboards.release-notes-1.3.10.md)
- [Opensearch dashboards.release notes 1.3.11](../release-notes/opensearch-dashboards.release-notes-1.3.11.md)
- [Opensearch dashboards.release notes 1.3.12](../release-notes/opensearch-dashboards.release-notes-1.3.12.md)
- [Opensearch dashboards.release notes 1.3.13](../release-notes/opensearch-dashboards.release-notes-1.3.13.md)
Copy link
Member

Choose a reason for hiding this comment

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

Nit: order need to adjust I suppose

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

Create CNAME

Delete CNAME

disable jekyll

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

change to relative path

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

add docs readme

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

Adds changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
@ashwin-pc ashwin-pc self-assigned this Mar 5, 2024
@kavilla
Copy link
Member

kavilla commented Mar 11, 2024

good stuff!

@ashwin-pc ashwin-pc merged commit 3aa0e5a into opensearch-project:main Mar 11, 2024
14 checks passed
@ashwin-pc ashwin-pc deleted the docs branch March 12, 2024 02:04
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 13, 2024
* Working Dev docs

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

Create CNAME

Delete CNAME

disable jekyll

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

change to relative path

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

add docs readme

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

Adds changelog

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

* removes unnecessary debugger

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
(cherry picked from commit 3aa0e5a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ashwin-pc pushed a commit that referenced this pull request Sep 13, 2024
* Working Dev docs



Create CNAME

Delete CNAME

disable jekyll



change to relative path



add docs readme



Adds changelog



* removes unnecessary debugger



---------


(cherry picked from commit 3aa0e5a)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants