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

Orchestra docs #2237

Merged
merged 16 commits into from
Feb 19, 2025
Merged

Orchestra docs #2237

merged 16 commits into from
Feb 19, 2025

Conversation

hugolu-1
Copy link
Contributor

Description

Related Issues

  • Fixes #...
  • Closes #...
  • Resolves #...

Additional Context

Copy link

netlify bot commented Jan 27, 2025

Deploy Preview for dlt-hub-docs ready!

Name Link
🔨 Latest commit 8c3232c
🔍 Latest deploy log https://app.netlify.com/sites/dlt-hub-docs/deploys/67b4b819b064c9000820480e
😎 Deploy Preview https://deploy-preview-2237--dlt-hub-docs.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.

adrianbr
adrianbr previously approved these changes Feb 12, 2025
@adrianbr
Copy link
Contributor

@dat-a-man can you add sidebars if needed?

@hugolu-1
Copy link
Contributor Author

@dat-a-man let me know if anything unclear in PR

@dat-a-man
Copy link
Collaborator

@hugolu-1 I'm trying to add sidebars, but I'm running into a permission error when pushing my changes:

@dat-a-man ➜ /workspaces/dlt (orchestra_docs) $ git push
remote: Permission to hugolu-1/dlt.git denied to dat-a-man.
fatal: unable to access 'https://github.com/hugolu-1/dlt/': The requested URL returned error: 403

Would you be able to grant me write access to your fork so I can push my changes? Let me know, thanks! 🙌

@hugolu-1
Copy link
Contributor Author

@dat-a-man should be ok try now?

image

…ppet titles (e.g., renamed python to py and yml to yaml) to align with the site's naming conventions.
@dat-a-man
Copy link
Collaborator

Yes, it's working now, @hugolu-1. I made a few minor adjustments. Thanks!

@adrianbr adrianbr self-requested a review February 17, 2025 13:37
@AstrakhantsevaAA AstrakhantsevaAA changed the base branch from devel to docs/orchestra-temp February 17, 2025 14:45
@adrianbr adrianbr changed the base branch from docs/orchestra-temp to devel February 18, 2025 16:39
@adrianbr adrianbr changed the base branch from devel to docs/orchestra-temp February 19, 2025 08:58
@adrianbr adrianbr merged commit 17c42ee into dlt-hub:docs/orchestra-temp Feb 19, 2025
52 checks passed
AstrakhantsevaAA added a commit that referenced this pull request Feb 19, 2025
* fix links (#1977)

* Adding Orchestra Docs Section

* Updated the sidebar to include new documentation and standardized snippet titles (e.g., renamed python to py and yml to yaml) to align with the site's naming conventions.

* Update docs/website/docs/walkthroughs/deploy-a-pipeline/deploy-with-orchestra.md



* Update docs/website/docs/walkthroughs/deploy-a-pipeline/deploy-with-orchestra.md



* Update docs/website/docs/walkthroughs/deploy-a-pipeline/deploy-with-orchestra.md



* Update docs/website/docs/walkthroughs/deploy-a-pipeline/deploy-with-orchestra.md



* Update docs/website/docs/walkthroughs/deploy-a-pipeline/deploy-with-orchestra.md



---------

Co-authored-by: Hugo Lu <[email protected]>
Co-authored-by: rudolfix <[email protected]>
Co-authored-by: David Scharf <[email protected]>
Co-authored-by: dat-a-man <[email protected]>
Co-authored-by: Alena Astrakhantseva <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants