Skip to content

DOCSP-13859: UpdateOne usage example #5

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
Merged
merged 11 commits into from
Jul 28, 2021
Merged

DOCSP-13859: UpdateOne usage example #5

merged 11 commits into from
Jul 28, 2021

Conversation

rustagir
Copy link
Contributor

@rustagir rustagir commented Jul 22, 2021 •
edited
Loading

Pull Request Info

Issue JIRA link:

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

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

https://docs-mongodbcom-staging.corp.mongodb.com/golang/docsworker-xlarge/DOCSP-13859-update-one-usage-ex/usage-examples/updateOne/

TODO: edit links to Fundamentals pages "Change a Document" and "Retrieve Data" when they are created

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?

@rustagir rustagir marked this pull request as draft July 22, 2021 21:07
@rustagir rustagir closed this Jul 23, 2021
@rustagir rustagir reopened this Jul 23, 2021
@rustagir rustagir marked this pull request as ready for review July 23, 2021 14:06
terakilobyte
terakilobyte requested changes Jul 23, 2021
View reviewed changes
Copy link
Member

@terakilobyte terakilobyte left a comment

Choose a reason for hiding this comment

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

Initial feedback

terakilobyte
terakilobyte approved these changes Jul 23, 2021
View reviewed changes
Copy link
Member

@terakilobyte terakilobyte left a comment

Choose a reason for hiding this comment

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

lgtm

@rustagir rustagir requested a review from kevinAlbs July 23, 2021 15:51
@rustagir
Copy link
Contributor Author

Hi @kevinAlbs, this PR is ready for you to review!

matthewdale
matthewdale reviewed Jul 26, 2021
View reviewed changes
@rustagir rustagir mentioned this pull request Jul 27, 2021
Merged
6 tasks
matthewdale
matthewdale approved these changes Jul 27, 2021
View reviewed changes
Copy link
Collaborator

@matthewdale matthewdale left a comment

Choose a reason for hiding this comment

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

👍

kevinAlbs
kevinAlbs reviewed Jul 27, 2021
View reviewed changes
Copy link
Collaborator

@kevinAlbs kevinAlbs left a comment

Choose a reason for hiding this comment

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

Looks great, I only have nits.

@kevinAlbs kevinAlbs mentioned this pull request Jul 28, 2021
Merged
6 tasks
@rustagir rustagir merged commit 1291c10 into mongodb:master Jul 28, 2021
@matthewdale matthewdale mentioned this pull request Jul 28, 2021
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthewdale matthewdale matthewdale approved these changes

@kevinAlbs kevinAlbs kevinAlbs approved these changes

@terakilobyte terakilobyte terakilobyte approved these changes

@benjirewis benjirewis benjirewis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rustagir @kevinAlbs @terakilobyte @matthewdale @benjirewis