Skip to content
This repository was archived by the owner on May 17, 2021. It is now read-only.

Conversation

imlucas
Copy link
Contributor

@imlucas imlucas commented Mar 19, 2020

  • fix(import): COMPASS-4206: Make type detection/casting work right when importing/exporting .json. a093933

Importing correctly picks up ObjectID, Number, and Date:

Screenshot 2020-03-19 17 15 49

  • feat(export): Pick up nested objects to include in projection f6baf06
doc = {
  _id: 'Arlo', 
  location: {
    place: 'home',
    activity: {
       sleeping: true,
       is: 'on the couch'
    }
};

Before this would populate the projection as:

  • _id
  • location

Now you can specify full paths for more control over what is actually returned:

  • _id
  • location.place
  • location.activity.is
  • location.activity.sleeping

Use case: (See a 1M line csv imported that is 300 fields wide and you only want 2).

  • fix(export): Add tests to make sure csv's have correct headers. df70ff9

Headers should be dotnotation for nested object paths.

@imlucas imlucas merged commit 00bd3c0 into master Mar 19, 2020
@imlucas imlucas deleted the objectid branch March 19, 2020 21:31
imlucas added a commit to mongodb-js/compass that referenced this pull request Mar 23, 2020
…1925)

* fix(import): COMPASS-4206: Fix type detection and export formatting.

mongodb-js/compass-import-export#144

* fix(import): COMPASS-4026: More fixes for importing EJSON

* chore(security): npm audit fix

fixed 121 of 256 vulnerabilities in 36223 scanned packages
imlucas added a commit to mongodb-js/compass that referenced this pull request Apr 2, 2020
…rmatting. (#1925)

* fix(import): COMPASS-4206: Fix type detection and export formatting.

mongodb-js/compass-import-export#144

* fix(import): COMPASS-4026: More fixes for importing EJSON

* chore(security): npm audit fix

fixed 121 of 256 vulnerabilities in 36223 scanned packages
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant