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: structure and content update #2875

Merged
merged 93 commits into from
Sep 11, 2024
Merged

docs: structure and content update #2875

merged 93 commits into from
Sep 11, 2024

Conversation

JoblersTune
Copy link
Collaborator

@JoblersTune JoblersTune commented Aug 20, 2024

Changes proposed in this pull request

  • Do not merge -> still a collaborative work in progress
  • Restructuring the docs and adding in updated content

Context

Checklist

  • Related issues linked using fixes #number
  • Tests added/updated
  • Documentation added
  • Make sure that all checks pass
  • Bruno collection updated

@github-actions github-actions bot added pkg: documentation Changes in the documentation package. type: documentation (archived) Improvements or additions to documentation labels Aug 20, 2024
Copy link

netlify bot commented Aug 20, 2024

Deploy Preview for brilliant-pasca-3e80ec ready!

Name Link
🔨 Latest commit 06d0a53
🔍 Latest deploy log https://app.netlify.com/sites/brilliant-pasca-3e80ec/deploys/66e0b8d1dae6220008cee870
😎 Deploy Preview https://deploy-preview-2875--brilliant-pasca-3e80ec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JoblersTune JoblersTune added the do not merge Do not merge PRs with these label label Aug 20, 2024
huijing and others added 10 commits August 21, 2024 22:16
- Intro > Overview
- Intro > ASE
- Integration > Getting Started
- Integration > Exchange Rates
- Integration > IDP
- Ran prettier to avoid the docs-pocalypse
Added the following pages:
- Integration > Adding asset
- Integration > Adding peer
- Integration > Creating wallet addresses
- Components > Accounts, transfers, and liquidity
- Resources > Glossary
- Resources > Admin UI Guide
- Adding contextual information around obtaining login credentials
- Adding information around deactivating a wallet address through the UI
README.md Outdated Show resolved Hide resolved
@JoblersTune
Copy link
Collaborator Author

The Rafiki Admin auth docs still need to be included. Currently we have a page packages/documentation/src/content/docs/rafikiadmin/auth.md which is not used. This content needs to be added in the correct location and the packages/documentation/src/content/docs/rafikiadmin folder must then be deleted as well.

@github-actions github-actions bot added pkg: backend Changes in the backend package. type: source Changes business logic labels Sep 10, 2024
- Incorporate Rafiki Admin auth information in the guide
- Removed auth.md
- Removed rafikiadmin folder
@brad-dow
Copy link
Contributor

The Rafiki Admin auth docs still need to be included. Currently we have a page packages/documentation/src/content/docs/rafikiadmin/auth.md which is not used. This content needs to be added in the correct location and the packages/documentation/src/content/docs/rafikiadmin folder must then be deleted as well.

I've removed auth.md as well as the rafikiadmin folder. All of the contents of the deleted auth page are now located in the Rafiki Admin guide.

@JoblersTune JoblersTune merged commit 0b92ce3 into main Sep 11, 2024
30 of 42 checks passed
@JoblersTune JoblersTune deleted the sj/docs-structure-update branch September 11, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge PRs with these label pkg: backend Changes in the backend package. pkg: documentation Changes in the documentation package. pkg: frontend Changes in the frontend package. pkg: mock-ase type: documentation (archived) Improvements or additions to documentation type: localenv Local playground type: source Changes business logic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants