Skip to content
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

[Suggestion]Change the notification "Cannot read property 'path' of undefined" to " The documents do not contain the partition key" #1581

Open
v-xinda opened this issue Jun 23, 2020 · 0 comments
Assignees
Labels
AT-CTI Created by CTI team bug The issue reveals something not working as expected. enhancement Small changes that can slightly improve user experiences. NoSQL P1

Comments

@v-xinda
Copy link

v-xinda commented Jun 23, 2020

OS: Mac
Build Version: 20200623.1

Repro step:

  1. Drill down on a Collection with partition key.
  2. Right click -->Click "Import Document into a Collection…".
  3. Select a .json file doesn't contain the partition key.

Actual:
Pop up an error notification "Cannot read property 'path' of undefined".
image

Suggestion:
Pop up an error notification "The documents do not contain the partition key".

More Info:

[]3:14:11 PM: The following documents do not contain the required partition key:
/Users/azuretoolstest/Desktop/Attachments for Cosmos DB/Reserves.json
3:14:11 PM: Error: Cannot read property 'path' of undefined
@v-xinda v-xinda added the AT-CTI Created by CTI team label Jun 23, 2020
@wwlorey wwlorey added this to the 0.15.0 milestone Jul 2, 2020
@neelip neelip modified the milestones: 0.16.0, 0.17.0 Dec 2, 2020
@ejizba ejizba modified the milestones: 0.17.0, Backlog Candidates Apr 5, 2021
@ejizba ejizba added P1 bug The issue reveals something not working as expected. labels Apr 5, 2021
@neelip neelip modified the milestones: 0.18.0, Backlog Candidates Jul 16, 2021
@sevoku sevoku added enhancement Small changes that can slightly improve user experiences. NoSQL labels Nov 14, 2024
@bk201- bk201- self-assigned this Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT-CTI Created by CTI team bug The issue reveals something not working as expected. enhancement Small changes that can slightly improve user experiences. NoSQL P1
Projects
None yet
Development

No branches or pull requests

6 participants