-
Notifications
You must be signed in to change notification settings - Fork 4
Feature: Easy Data Migrations (Collection Copy-and-Paste) #170
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
base: next
Are you sure you want to change the base?
Conversation
Co-authored-by: tnaum-ms <[email protected]>
Co-authored-by: tnaum-ms <[email protected]>
Co-authored-by: tnaum-ms <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
…aste-2-implement-basic-copy-and-paste-task
Co-authored-by: Copilot <[email protected]>
…-documentdb into dev/xingfan/111-copy-and-paste-2-implement-basic-copy-and-paste-task
…into dev/xingfan/111-copy-and-paste-2-implement-basic-copy-and-paste-task
|
@copilot This branch is out-of-date with the base branch. Merge the latest changes from next into this branch. Resolve conflicts: |
d1706ce to
b9346e3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 52 out of 54 changed files in this pull request and generated 1 comment.
| if (targetNode) { | ||
| context.telemetry.properties.targetNodeProperties = Object.getOwnPropertyNames(targetNode).join(','); | ||
| context.telemetry.properties.targetNodeHasCluster = String('cluster' in targetNode); | ||
| context.telemetry.properties.targetNodeHasDatabaseInfo = String('databaseInfo' in targetNode); | ||
| context.telemetry.properties.targetNodeHasCollectionInfo = String('collectionInfo' in targetNode); | ||
| } |
Copilot
AI
Nov 27, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This use of variable 'targetNode' always evaluates to true.
| if (targetNode) { | |
| context.telemetry.properties.targetNodeProperties = Object.getOwnPropertyNames(targetNode).join(','); | |
| context.telemetry.properties.targetNodeHasCluster = String('cluster' in targetNode); | |
| context.telemetry.properties.targetNodeHasDatabaseInfo = String('databaseInfo' in targetNode); | |
| context.telemetry.properties.targetNodeHasCollectionInfo = String('collectionInfo' in targetNode); | |
| } | |
| context.telemetry.properties.targetNodeProperties = Object.getOwnPropertyNames(targetNode).join(','); | |
| context.telemetry.properties.targetNodeHasCluster = String('cluster' in targetNode); | |
| context.telemetry.properties.targetNodeHasDatabaseInfo = String('databaseInfo' in targetNode); | |
| context.telemetry.properties.targetNodeHasCollectionInfo = String('collectionInfo' in targetNode); |
Easy Data Migrations for DocumentDB and MongoDB databases is a feature designed to simplify small-scale data migrations by leveraging a user-friendly copy-and-paste experience. This feature is ideal for smaller datasets where all data can be moved through the user's local machine.
For details of this PR review: #63
closes #63