Alaa_Nahser-w3-databases #29
Conversation
Week3/homework/mongodb/index.js
Outdated
| .db("databaseWeek3") | ||
| .collection("bob_ross_episodes") | ||
| .updateMany({ elements: "BUSHES" }, [ | ||
| { |
There was a problem hiding this comment.
Your first argument to updateMany is fine, but your second argument is more complex than it needs to be. Can you find a way to simplify this? Try using the all-positional operator: https://www.mongodb.com/docs/manual/reference/operator/update/positional-all/
crevulus
left a comment
crevulus
left a comment
There was a problem hiding this comment.
Hi Alaa,
Your code is great! I've already approved, but I have left one comment. You might want to consider making your code simpler and leveraging existing utilities and functionality provided by the mongodb package.
crevulus
left a comment
crevulus
left a comment
There was a problem hiding this comment.
Approved PR by mistake (instead of just changing the label) - please do not merge the code.
|
I have edited the indes.js file. I used the all positional operator $[]. I hope it is good now. Thank you. |
|
2 participants
No description provided.