-
Notifications
You must be signed in to change notification settings - Fork 112
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
Integrate FastAPI with Website, Restructure Project, and Enhance Compatibility #386
base: main
Are you sure you want to change the base?
Integrate FastAPI with Website, Restructure Project, and Enhance Compatibility #386
Conversation
…lders, and update requirements.txt
🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from team will review it soon. We appreciate your support in making better. |
✅ Deploy Preview for multiverse-dataverse ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@Tejas-Gupta-7 Just a heads-up: since this PR changes the project structure, other PRs related to the website might not merge as expected due to incorrect file paths. |
Hi @Rucha-Ambaliya, I cloned the PR and tried to test it but I'm facing some issues.
Do you know what the issue is? PS: I've put some of the PRs and issues on I would appreciate hearing your thoughts on this. Also as the issue #374 demands greater time and efforts, I've decided to merge a dummy PR to assign you appropriate points. So, it would be nice if you could create a blank PR and refer to this comment in it. 👍 |
Changes MadeI have made two changes in
After making these changes, I forked the branch Steps Followed
Both the app and website are now running properly on my system. Debugging the IssueI faced the same issue where the modules were listed as installed, but I was still getting the "module not found" error. In my case, the issue was that I hadn't activated the virtual environment. In your case, it seems like you installed the dependencies globally. Could you try the steps I mentioned above, ensuring the virtual environment is activated? This will help me compare and identify the issue more clearly. Regarding the PRThank you for putting the other PRs on hold. This will help me avoid conflicts when merging. The idea of completing the entire issue in a single PR and creating a dummy PR sounds good to me. This approach will help avoid any further conflicts and also give us an opportunity to properly test FastAPI functionality. I’ll continue working on the changes related to the model in this same branch. Thank you for your cooperation and patience @Tejas-Gupta-7! |
Related Issue
Partially Fixes #374
Description
This pull request focuses on integrating FastAPI with the existing website, restructuring the project for better maintainability, and ensuring seamless compatibility with MongoDB. The updates also include improved documentation and dependency management.
Key Updates
1. Dependency Management
requirements.txt
to include the latest dependencies, ensuring the project runs smoothly.2. Enhanced Project Structure
PROJECT_STRUCTURE.md
to reflect the new organization.3. FastAPI Integration
main.py
inwebsite
folder to serve all HTML pages dynamically:4. Static File Routing:
5. MongoDB Integration:
6. Secure Environment Management:
.env
file support to securely manage sensitive configurations like database URLs.7. Documentation Improvements:
README.md
to include steps for running the website using FastAPI.Testing
Screenshots
Live website:
Recording.2025-01-17.202140.1.mp4
main.py in website
Recording.2025-01-17.233425.mp4
README.md
PROJECT_STRUCTURE.md
Please review the changes and share your feedback. If any additional enhancements or modifications are needed, let me know!
Type of change
Checklist