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

Fix development build #68

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

adamgoth
Copy link

@adamgoth adamgoth commented May 21, 2024

I encountered several errors trying to build the app on the development branch. This PR addresses those.

  • Update webpack module rules to fix webpack build failures
  • Fixes early terminated string in prompts.ts
  • Fixes incorrect name of function imports from ./rag in ollama.ts
  • Fixes type error when creating chain in askOllamaRAG function
  • Updates parsing of json string response in frontend/utils/utils.ts
  • Handles errors when parsing JSON

The development branch should now build without errors and correctly display a response from ollama.

@adamgoth
Copy link
Author

I saw the Windows build failed again. It looks related to this line: https://github.com/MorpheusAIs/Lite-Client/pull/51/files

It appears the intention was to make loadingGif undefined, but instead a string value of 'undefined' was used, so it's looking for a gif by that name that doesn't exist. I pushed an update to remove the single quotes, which should hopefully fix the build.

@adamgoth
Copy link
Author

Looks like a network issue of some kind in the latest run

Screenshot 2024-05-29 at 9 21 58 PM

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.

1 participant