yoheinakajima / instagraph

Converts text input or URL into knowledge graph and displays
MIT License
3.27k stars 281 forks source link

WIP: Fixes a few bugs #49

Closed yoheinakajima closed 10 months ago

yoheinakajima commented 11 months ago

This fixes some bugs I had with neo4j turned on, but I didn't really clean it up. Since I don't know if I'll have a chance to touch code for a week, dropping it here in case people want to look and fix from here. This version pulls the entire knowledge graph so will combine the results of all the queries in the display.

Summary by CodeRabbit

Release Notes:

coderabbitai[bot] commented 11 months ago

Walkthrough

The changes primarily focus on improving the code quality and user experience of a web application that interacts with a Neo4j database. The Python backend has been enhanced with better error handling and query optimization, while the frontend has seen significant refactoring for improved UI responsiveness, simplified functions, and better error display.

Changes

File(s) Summary
main.py Improved exception handling from neo4j.exceptions, optimized Neo4j queries, and enhanced JSON data processing.
templates/index.html Refactored JavaScript functions for simplicity and efficiency, improved form submission and error handling, and updated UI for better responsiveness across different screen sizes.

🐇💻

Code refined, errors declined,

With each line, the app's design shines.

From backend to front, improvements are blunt,

Celebrate we must, for this is no minor thrust! 🎉🥕

Tips ### Chat with CodeRabbit Bot (`@coderabbitai`) - Mention `@coderabbitai` in any *review comment* for bot assistance. - Note: Review comments are made on code diffs or files, not on the PR overview. ### Pause Incremental Reviews - Insert `@coderabbitai: ignore` in the PR description to halt the bot's ongoing reviews. Remove the line to resume.
shepherd-06 commented 11 months ago

I will look into it today/tomorrow. @yoheinakajima

snakajima commented 10 months ago

Json sanitization and improved neo4jquery are done as PR59 and PR66. The rest of changes are no longer needed because we have Vue-based UI. Closing.