-
Notifications
You must be signed in to change notification settings - Fork 14
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
Additional updates for microsoft fabric documentation #8618
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@asimonutti do you want to see this?
_docs/_user_guide/data_and_analytics/cloud_ingestion/connected_sources.md
Outdated
Show resolved
Hide resolved
_docs/_user_guide/data_and_analytics/cloud_ingestion/connected_sources.md
Outdated
Show resolved
Hide resolved
_docs/_user_guide/data_and_analytics/cloud_ingestion/connected_sources.md
Outdated
Show resolved
Hide resolved
Changes look good to me aside from minor nitpicks! Holding off in case @asimonutti has edits. Will merge before EOD 12/12. |
Taking a look now! |
LGTM |
Why are you making this change? (required)
Adding documentation for other CDI functionality supported by Azure Fabric
Related PRs, issues, or features (optional)
Feature release date (optional)
This feature is in EA on 12/12. If these changes can go out with that docs release that would be great - otherwise, the following release is ok.
Contributor checklist
Submitting for review
If your PR meets the above requirements, select Ready for review, then add a reviewer:
braze-inc/docs-team
as the reviewer.braze-inc/docs-team
instead.Thanks for contributing! We look forward to reading your work.