Skip to content

DOCSP-13826 landing page #17

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 3 commits into from
Aug 6, 2021

Conversation

kyuan-mongodb
Copy link
Contributor

@kyuan-mongodb kyuan-mongodb commented Aug 5, 2021

Pull Request Info

Added the landing page.

Utilized the formatting of the Node and Java docs.

Note, I commented out sections (What's New, FAQ, other fundamental sections) that aren't planned to be in the first release of the docs.

Issue JIRA link:

https://jira.mongodb.org/browse/DOCSP-13826

Docs staging link (requires sign-in on MongoDB Corp SSO):

https://docs-mongodbcom-staging.corp.mongodb.com/golang/docsworker-xlarge/DOCSP-13826-LandingPage/

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?
  • Does it render on staging correctly?
  • Are all the links working?
  • Are the staging links in the PR description updated?

Copy link

@sarahsimpers sarahsimpers left a comment

Choose a reason for hiding this comment

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

Just a few small suggestions to simplify wording and improve readability. LGTM!

@kyuan-mongodb kyuan-mongodb changed the title DOCSP-1382 landing page DOCSP-13826 landing page Aug 5, 2021
Copy link
Contributor

@rustagir rustagir left a comment

Choose a reason for hiding this comment

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

A few small fixes and questions, but looks good!

@kyuan-mongodb kyuan-mongodb merged commit 133dd62 into mongodb:master Aug 6, 2021
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