Skip to content

Conversation

shuangela
Copy link
Collaborator

@shuangela shuangela commented Mar 3, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-46756
Staging - https://deploy-preview-437--docs-golang.netlify.app/fundamentals/crud/read-operations/changestream/

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 Mar 3, 2025

Deploy Preview for docs-golang ready!

Name Link
🔨 Latest commit bc2e102
🔍 Latest deploy log https://app.netlify.com/sites/docs-golang/deploys/67c730f48f20bd0008e319ea
😎 Deploy Preview https://deploy-preview-437--docs-golang.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
Collaborator

@norareidy norareidy left a comment

Choose a reason for hiding this comment

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

LGTM!

@shuangela shuangela requested review from sarahemlin and removed request for sarahemlin March 3, 2025 20:11
Copy link
Collaborator

@lindseymoore lindseymoore left a comment

Choose a reason for hiding this comment

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

Made some title edits, let me know if they make sense to you.

Monitor Data Changes
====================
============================
Monitor Data Changes Example
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Monitor Data Changes Example
Open a Change Stream Example

S: Discussed this title with Sarah. I would change this title to include Change Stream, as the page titles for pages within the Usage Example section trend towards being more product or method name focused. "Open a Change Stream" will do to ensure page title meets length requirements.

Write Operations </usage-examples/write-operations>
Bulk Operations </usage-examples/bulkWrite>
Monitor Data Changes </usage-examples/changestream>
Monitor Data Changes Example </usage-examples/changestream>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Monitor Data Changes Example </usage-examples/changestream>
Open a Change Stream </usage-examples/changestream>

Copy link
Collaborator

@lindseymoore lindseymoore left a comment

Choose a reason for hiding this comment

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

LGTM! @sarahemlin would you like to take a look?

@lindseymoore lindseymoore requested a review from sarahemlin March 4, 2025 17:00
Copy link
Collaborator

@sarahemlin sarahemlin left a comment

Choose a reason for hiding this comment

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

Thanks for including 'change stream' in the title so that it matches how we talk about it in the Manual!

@shuangela
Copy link
Collaborator Author

💚 All backports created successfully

Status Branch Result
v2.0
v1.17
v1.16
v1.15
v1.14
v1.13
v1.12

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

shuangela added a commit that referenced this pull request Mar 10, 2025
[v2.0] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
shuangela added a commit that referenced this pull request Mar 10, 2025
[v1.17] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
shuangela added a commit that referenced this pull request Mar 10, 2025
[v1.16] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
shuangela added a commit that referenced this pull request Mar 10, 2025
[v1.15] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
shuangela added a commit that referenced this pull request Mar 10, 2025
[v1.14] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
shuangela added a commit that referenced this pull request Mar 10, 2025
[v1.13] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
shuangela added a commit that referenced this pull request Mar 10, 2025
[v1.12] Merge pull request #437 from shuangela/DOCSP-46756-monitor-changes-title
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.

4 participants