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 several dead-links #807

Merged
merged 5 commits into from
Jan 8, 2024
Merged

Fix several dead-links #807

merged 5 commits into from
Jan 8, 2024

Conversation

badboy
Copy link
Member

@badboy badboy commented Jan 8, 2024

This fixes some dead links and moves the activity stream dataset docs into obsolete.

@badboy badboy requested review from chutten and ANich January 8, 2024 14:32
@@ -151,10 +150,11 @@
- [Metrics Standardization and Policy](metrics/policy.md)
- [Legacy Census Metrics](concepts/censuses.md)
- [Obsolete Datasets](datasets/obsolete.md)
- [Activity Stream](datasets/obsolete/activity-stream/reference.md)
Copy link
Member Author

Choose a reason for hiding this comment

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

@chutten as promised the move here.

Comment on lines 155 to +157
- [Churn](datasets/obsolete/churn/reference.md)
- [Client Count](datasets/obsolete/client_count/reference.md)
- [Client Count Daily](datasets/obsolete/client_count_daily/reference.md)
- [Client Count](datasets/obsolete/client_count/reference.md)
Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, I re-ordered those things by alphabet.

@@ -6,8 +6,7 @@ Instead, identify a dataset you'd like to understand better and read through
the relevant documentation.
After reading the tutorial, you should know all you need about the dataset.

Also see more detailed [per-table docs](https://mozilla.github.io/bigquery-etl/mozdata/introduction/)
that are generated alongside ETL code in the `bigquery-etl` repository.
Detailed per-table docs are available [in the Data Catalog](https://mozilla.acryl.io/).
Copy link
Member Author

Choose a reason for hiding this comment

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

@ANich ok like this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah this is okay, I'm going to write a longer page on this so I might update this in that PR 👍

@@ -6,8 +6,7 @@ Instead, identify a dataset you'd like to understand better and read through
the relevant documentation.
After reading the tutorial, you should know all you need about the dataset.

Also see more detailed [per-table docs](https://mozilla.github.io/bigquery-etl/mozdata/introduction/)
that are generated alongside ETL code in the `bigquery-etl` repository.
Detailed per-table docs are available [in the Data Catalog](https://mozilla.acryl.io/).
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah this is okay, I'm going to write a longer page on this so I might update this in that PR 👍

@badboy badboy merged commit 2f677e2 into main Jan 8, 2024
9 checks passed
@badboy badboy deleted the fix-dead-links-again branch January 8, 2024 16:38
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.

3 participants