Skip to content

[Java Sync] P2 - Shift Java ToC #617

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

Merged
merged 6 commits into from
Feb 12, 2025

Conversation

rachel-mack
Copy link
Contributor

@rachel-mack rachel-mack commented Feb 11, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-47038

Part 2: Level 2 organization

Change the side bar drop downs to match the universal TOC, including re-adding topics that were removed during the L1 reorg.

NOTE

I will be ignoring language-related Vale errors, as these errors are pre-existing and outside the scope of this project.

Staging Links

https://deploy-preview-617--docs-java.netlify.app/

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?
  • Are the facets and meta keywords accurate?

Copy link

netlify bot commented Feb 11, 2025

Deploy Preview for docs-java ready!

Name Link
🔨 Latest commit e6ff6da
🔍 Latest deploy log https://app.netlify.com/sites/docs-java/deploys/67acf0184ba5fe000867ca41
😎 Deploy Preview https://deploy-preview-617--docs-java.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@mcmorisi mcmorisi left a comment

Choose a reason for hiding this comment

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

LGTM with a couple of fixes!

@@ -1,8 +1,8 @@
.. _java-sync-quickstart:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you need to add this page to this list of toc_landing_pages in the snooty.toml, otherwise you can't click into this page from the ToC sidebar.

@@ -6,12 +6,12 @@ Builders

.. toctree::

Aggregation </builders/aggregates>
Copy link
Contributor

Choose a reason for hiding this comment

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

This page also needs to be added to list of toc_landing_pages

@rachel-mack rachel-mack merged commit 52989b6 into mongodb:master Feb 12, 2025
6 checks passed
rachel-mack added a commit to rachel-mack/docs-java that referenced this pull request Mar 24, 2025
rachel-mack added a commit to rachel-mack/docs-java that referenced this pull request Mar 24, 2025
rachel-mack added a commit to rachel-mack/docs-java that referenced this pull request Mar 24, 2025
rachel-mack added a commit to rachel-mack/docs-java that referenced this pull request Mar 24, 2025
rachel-mack added a commit that referenced this pull request Mar 24, 2025
* [Java Sync] P1 - Shift Java ToC (#615)

* fix redirects (#616)

* [Java Sync] P2 - Shift Java ToC  (#617)

* toc tweak

* force build

* aggregates

* redirects

* reference

* toc

* databases adn collections
rachel-mack added a commit that referenced this pull request Mar 24, 2025
* [Java Sync] P1 - Shift Java ToC (#615)

* fix redirects (#616)

* [Java Sync] P2 - Shift Java ToC  (#617)

* toc-tweaks

* redirects

* reference

* toc

* databases adn collections

* reference links

* rm reference landing
rachel-mack added a commit that referenced this pull request Mar 24, 2025
* [Java Sync] P1 - Shift Java ToC (#615)

* fix redirects (#616)

* [Java Sync] P2 - Shift Java ToC  (#617)

* toc tweak

* reference

* databases adn collections

* reference links

* rm reference landing
rachel-mack added a commit that referenced this pull request Mar 24, 2025
* [Java Sync] P1 - Shift Java ToC (#615)

* fix redirects (#616)

* [Java Sync] P2 - Shift Java ToC  (#617)

* toc tweaks

* databases adn collections

* rm reference landing

* vale error
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.

2 participants