Skip to content

Replace .push with .concat to port to Mongodb 4, fixes Unknown modifier: $pushAll#116

Open
cirosantilli wants to merge 3 commits intogothinkster:masterfrom
cirosantilli:master
Open

Replace .push with .concat to port to Mongodb 4, fixes Unknown modifier: $pushAll#116
cirosantilli wants to merge 3 commits intogothinkster:masterfrom
cirosantilli:master

Conversation

@cirosantilli
Copy link
Copy Markdown

@cirosantilli cirosantilli commented Mar 22, 2021

Otherwise actions such as commenting fail with:

Unknown modifier: $pushAll

As per: https://stackoverflow.com/questions/48607918/mongoerror-unknown-modifier-pushall-in-node-js/50435618#50435618

Tested on Ubuntu 20.10, Mongo 4.4.4, https://github.com/gothinkster/vue-realworld-example-app
at 61742206c170db02b04d63c7e9d43807d8c6b902 with only the URL modified:

diff --git a/src/common/config.js b/src/common/config.js
index 03af84e..ccd9c4d 100644
--- a/src/common/config.js
+++ b/src/common/config.js
@@ -1,2 +1,2 @@
-export const API_URL = "https://conduit.productionready.io/api";
+export const API_URL = "http://localhost:3000/api";
 export default API_URL;

Fix #57
Fix #96
Fix #64

@cirosantilli cirosantilli changed the title Replace .push with .concat to port to Mongodb 4 Replace .push with .concat to port to Mongodb 4, fixes Unknown modifier: $pushAll Mar 22, 2021
Otherwise actions such as commenting fail with:

Unknown modifier: $pushAll

As per: https://stackoverflow.com/questions/48607918/mongoerror-unknown-modifier-pushall-in-node-js/50435618#50435618

Tested on Ubuntu 20.10, Mongo 4.4.4, https://github.com/gothinkster/vue-realworld-example-app
at 61742206c170db02b04d63c7e9d43807d8c6b902 with only the URL modified:

diff --git a/src/common/config.js b/src/common/config.js
index 03af84e..ccd9c4d 100644
--- a/src/common/config.js
+++ b/src/common/config.js
@@ -1,2 +1,2 @@
-export const API_URL = "https://conduit.productionready.io/api";
+export const API_URL = "http://localhost:3000/api";
 export default API_URL;

Fix gothinkster#57
Fix gothinkster#96
Fix gothinkster#64
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.

deprecated push method Unknown modifier: $pushAll Unknown modifier: $pushAll"

1 participant