Skip to content

Conversation

josmanperez
Copy link

  • [Fixed] When running the app following the README.md, there was an error thrown: Responding with 500 status and error message: invalid parameter: expected an object (filter).
  • [Improved] Added a small method to re-load the .env file in case something changes after running the application for the first time.

* Added a small method to re-load the  file in case something changes after running the application for the first time
package.json Outdated
Comment on lines 17 to 19
},
"dependencies": {
"mongodb-chatbot-server": "^0.9.1"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi, i'm not sure if this is the right place. think it should be in the packages' package.json files.
such as - https://github.com/mongodb-university/cookbook-chat/blob/main/packages/ingest/package.json#L13

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mongodben , I don't see a package.json inside the directory packages. The only one I could find is in the root directoy and adding that line and installing the packages it make the error go away.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right but inside the subdirectories of packages, like packages/ingest (as in the above link), you find the package.json files. in this case, packages/server/package.json should be updated

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

once you add to the correct location, remove from the root package.json, all should be good. though it's worth doing a quick validation by running the server

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @mongodben , I believe this is now done. Please let me know if you need something from my side.

Copy link
Collaborator

@mongodben mongodben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for this!

@mongodben mongodben merged commit 7c73e5f into mongodb-university:main Aug 21, 2024
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.

3 participants