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

Docs - Replacing Slack References #3772

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Docs - Replacing Slack References #3772

wants to merge 11 commits into from

Conversation

amandaha8
Copy link
Contributor

Description

We are moving to Teams to discuss all Caltrans related issues this summer. As such, any slack references in the docs need to be altered, where appropriate.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Sorry, something went wrong.

Copy link
Member

@tiffanychu90 tiffanychu90 left a comment

Choose a reason for hiding this comment

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

The jupyterbook build isn't working because it's looking for intro.md. Can you fix where that reference is either by linking it or removing it from the _toc.yml if the page is gone

@amandaha8
Copy link
Contributor Author

The jupyterbook build isn't working because it's looking for intro.md. Can you fix where that reference is either by linking it or removing it from the _toc.yml if the page is gone

I'll do that. Strange, I didn't remove any pages when updating!

@amandaha8
Copy link
Contributor Author

intro.md is still here (I didn't delete it out) and it looks like it's being referenced here in the toc.yml so I'm not sure what I need to do to make the build work

@@ -13,6 +13,7 @@
:::
**Collaboration Tools:**

- [ ] Microsoft Teams
- [ ] [**Analytics Repo**](https://github.com/cal-itp/data-analyses) | ([Docs](analytics-repo))
Copy link
Member

Choose a reason for hiding this comment

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

I'll spend more time with this code Monday but it looks like the links to other docs pages are broken - that's causing the build to error: /home/runner/work/data-infra/data-infra/docs/analytics_onboarding/overview.md:17: WARNING: 'myst' cross-reference target not found: 'analytics-repo' [myst.xref_missing]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks Katrina, I'm having trouble figuring out what is wrong.

Copy link
Member

Choose a reason for hiding this comment

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

This is what is looks like when I'm looking at main, but the Docs links are internal references that are supposed to link to other pages inside of the same subfolder that the page itself is located in. Right now those pages don't exist or are in some other folder?
image

Copy link
Member

Choose a reason for hiding this comment

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

that being said, idk why this doesn't present a problem with the link behavior on the actual published docs page

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@KatrinaMKaiser It built! I changed the link. I don't know how this never presented a problem, a page for analytics-repo never existed in the first place.

Copy link
Member

Choose a reason for hiding this comment

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

Can you confirm that the other Docs links on that page work and go to the correct pages?

Copy link


## How Cal-ITP relates to DDS

- Copy here.
Copy link
Member

Choose a reason for hiding this comment

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

Were we waiting on something from @edasmalchi here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@KatrinaMKaiser Yes! I had forgotten to remind Eric.

@@ -13,6 +13,7 @@
:::
**Collaboration Tools:**

- [ ] Microsoft Teams
- [ ] [**Analytics Repo**](https://github.com/cal-itp/data-analyses) | ([Docs](analytics-repo))
Copy link
Member

Choose a reason for hiding this comment

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

Can you confirm that the other Docs links on that page work and go to the correct pages?

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

3 participants