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

fix: make next as default version & create redirection #10309

Merged
merged 12 commits into from
Apr 19, 2024

Conversation

yoonhyejin
Copy link
Collaborator

Checklist

  • The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
  • For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@github-actions github-actions bot added the docs Issues and Improvements to docs label Apr 17, 2024
@yoonhyejin
Copy link
Collaborator Author

this should be merged first : datahub-project/static-assets#81

@hsheth2 hsheth2 added the merge-pending-ci A PR that has passed review and should be merged once CI is green. label Apr 17, 2024
@@ -10,7 +10,7 @@ import FeatureAvailability from '@site/src/components/FeatureAvailability';
# Configuring User/Group/Roles provisioning from MS Entra to DataHub

1. **Generate Personal Access Token**:
Generate a personal access token from [DataHub](https://datahubproject.io/docs/next/authentication/personal-access-tokens/#creating-personal-access-tokens).
Generate a personal access token from [DataHub](/docs/authentication/personal-access-tokens.md#creating-personal-access-tokens).
Copy link
Collaborator

Choose a reason for hiding this comment

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

let's make this a relative link?

@@ -430,7 +430,7 @@
}
},
{
"Path": "docs/generated/ingestion/sources/presto-on-hive",
"Path": "docs/generated/ingestion/sources/hive-metastore",
"imgPath": "img/logos/platforms/presto.svg",
"Title": "Presto on Hive",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"Title": "Presto on Hive",
"Title": "Hive Metastore",

@yoonhyejin yoonhyejin merged commit f9dbe98 into master Apr 19, 2024
51 of 52 checks passed
@yoonhyejin yoonhyejin deleted the docs/next-default branch April 19, 2024 06:12
sleeperdeep pushed a commit to sleeperdeep/datahub that referenced this pull request Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issues and Improvements to docs merge-pending-ci A PR that has passed review and should be merged once CI is green.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants