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.

Sign up for GitHub

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

Jump to bottom

feat: add native versioning on docs #8512

Closed
wants to merge 149 commits into from
Closed

feat: add native versioning on docs #8512

wants to merge 149 commits into from

Conversation

yoonhyejin
Copy link
Collaborator

@yoonhyejin yoonhyejin commented Jul 26, 2023 •
edited
Loading

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

We might have to trigger the algolia crawler manually after deployment, to enable versioning search

@github-actions github-actions bot added ingestion PR or Issue related to the ingestion of metadata docs Issues and Improvements to docs product PR or Issue related to the DataHub UI/UX devops PR or Issue related to DataHub backend & deployment labels Jul 26, 2023
@hsheth2
Copy link
Collaborator

hsheth2 commented Jul 28, 2023

I'm not a huge fan of the amount of copying here - would it be possible for the entire versioned_docs/version-0.10.4 directory to live in the static assets repo as a tar.gz file, and then pull it in during the docs build process?

kkorchak and others added 25 commits August 24, 2023 16:32
@kkorchak @yoonhyejin
test(): Lineage column level test (#8641)
6c41f35
@eboneil @yoonhyejin
tests(search): search golden tests (#8605)
65fd05b
@kkorchak @jjoyce0510 @yoonhyejin
test(): Add test case for dataset deprecation test (#8646)
d2dca21 
Co-authored-by: John Joyce <[email protected]>
@mayurinehate @yoonhyejin
docs(ingest/kafka-connect): add details on platform instance mapping (#…
98d0c86 
…8654)
@hsheth2 @yoonhyejin
docs(ingest/airflow): add capture_executions to docs (#8662)
031ae72
@yoonhyejin
Fix a few view select issues (#8670)
20e179c
@iprentic @yoonhyejin
feat(search): Add word gram analyzer for name fields (#8611)
349b88c 
Co-authored-by: Indy Prentice <[email protected]>
@david-leifker @yoonhyejin
fix(docker): misc docker fixes (#8677)
75e9d0e
@eboneil @yoonhyejin
tests(search): more golden tests (#8683)
d5e7539
@asikowitz @yoonhyejin
test(ingest/vertica): Skip integration test failing CI; support arm M…
7a1ffca 
…acs (#8694)
@hsheth2 @yoonhyejin
ci: add needs_artifact_download output for ingestion image (#8695)
2ee8932
@asikowitz @yoonhyejin
logs(ingestion/unity): Hide stack trace on sql parse failure logs (#8657
e2a1525 
)
@siddiquebagwan-gslab @yoonhyejin
feat(ingestion/powerbi): support multiple tables as upstream in nativ…
23ac952 
…e SQL parsing (#8592)
@asikowitz @yoonhyejin
build(ingest): Bump pydantic pin (#8660)
38f2718
@asikowitz @treff7es
@anshbansal @yoonhyejin
remove(ingest/snowflake): Remove legacy snowflake lineage (#8653)
2d09979 
Co-authored-by: Tamas Nemeth <[email protected]>
Co-authored-by: Aseem Bansal <[email protected]>
@asikowitz @yoonhyejin
fix(ingest/ldap): Handle case when 'objectClass' not in attrs (#8658)
4ee6d38
@chriscollins3456 @iprentic
@anshbansal @yoonhyejin
fix(ui) Remove new Role entity from searchable entity types (#8655)
c73b767 
Co-authored-by: Indy Prentice <[email protected]>
Co-authored-by: Aseem Bansal <[email protected]>
@chriscollins3456 @yoonhyejin
fix(java) Use alias for name search sorting and fix missing mappings (#…
7bc821e 
…8648)
@chriscollins3456 @yoonhyejin
feat(ui) Create page for managing home page posts (#8707)
fe7ffb4
@hsheth2 @yoonhyejin
fix(ingest/powerbi): add sqlglot python dep (#8704)
3056b10
@hsheth2 @yoonhyejin
ci(ingest): make ingestion caching rules correct (#8685)
663195c
@yoonhyejin
fix: fix etnernal image url
f72f701
@yoonhyejin
fix: add scripts to download the historical versions
1edc83d
@yoonhyejin
feat: add new version
cd20e8d
@yoonhyejin
fix: fix typo
85943ca
@yoonhyejin yoonhyejin marked this pull request as draft August 24, 2023 11:48
@hsheth2
Copy link
Collaborator

hsheth2 commented Aug 26, 2023

Closing this as outdated.

@hsheth2 hsheth2 closed this Aug 26, 2023
@yoonhyejin yoonhyejin deleted the docs/versioning-static-asset branch December 11, 2023 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@treff7es treff7es Awaiting requested review from treff7es

@hsheth2 hsheth2 Awaiting requested review from hsheth2

@jjoyce0510 jjoyce0510 Awaiting requested review from jjoyce0510

Assignees
No one assigned
Labels
devops PR or Issue related to DataHub backend & deployment docs Issues and Improvements to docs ingestion PR or Issue related to the ingestion of metadata product PR or Issue related to the DataHub UI/UX
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.