Skip to content

Add FastAPI Dashboard + WebSocket + DB Persistence#47

Merged
MuddyHope merged 9 commits into
MuddyHope:mainfrom
Vamsi-aki:feature/dashboard-websocket-db
Jul 1, 2025
Merged

Add FastAPI Dashboard + WebSocket + DB Persistence#47
MuddyHope merged 9 commits into
MuddyHope:mainfrom
Vamsi-aki:feature/dashboard-websocket-db

Conversation

@Vamsi-aki

Copy link
Copy Markdown
Contributor

This PR addresses the following issues:

All changes are modular and driven by config.py settings.
Screenshot From 2025-06-28 18-01-36

@MuddyHope

MuddyHope commented Jun 29, 2025 via email

Copy link
Copy Markdown
Owner

@MuddyHope

Copy link
Copy Markdown
Owner

HI @Vamsi-aki, can you make sure the test PR and lintings are passing?? Also can you make sure the dashboard is more optimized? Thank you.

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope, I've made the requested changes — optimized the dashboard, verified that tests and linting are passing. Let me know if anything else is needed. Thanks!

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope, I've fixed the linting issues and all style, tests checks are passing now. Please make sure sqlalchemy is installed via the updated requirements.txt. Thank you.

@MuddyHope

Copy link
Copy Markdown
Owner

@Vamsi-aki, can you check the tests again?

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope , all the issues have been resolved and checks are passing. Could you please review it again. Thanks!

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope, Fixed issue between flake8 and black. Updated requirements.txt . Could you please review it again? Thanks!

@MuddyHope

Copy link
Copy Markdown
Owner

Hi @Vamsi-aki,
Could you please run make lint to ensure everything is clean? Also, could you update the README to reflect your script—specifically, how someone can use the decorator and run the server alongside it?

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope , Working on decorator files will update and make another commit. Thanks!

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope, Fixed lint issues and updated README, Please let me know if there are any issues or further changes needed. Thank you!

@MuddyHope

Copy link
Copy Markdown
Owner

Hi @Vamsi-aki,

The commands in your readme doesn't match the actual commands.
uvicorn pyquerytracker.api:app --reload

Firstly, I don't see anything in the dashboard being reflected from the DB.
The DB is working fine. All the queries are being inserted into the DB, but can you ensure the calls to the server are handled accurately. I'm happy to jump on a call if you are having any issues committing. Thanks.

@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope,

-- Fixed Dashboard
-- Updated README
-- All checks passed (make lint, pytest)

Please let me know if there are any issues or further changes needed. Thank you!

@MuddyHope

Copy link
Copy Markdown
Owner

Let me test it @Vamsi-aki.

@MuddyHope

Copy link
Copy Markdown
Owner

Looks good @Vamsi-aki, but in the dashboard can you show the function name also? Instead of slow or normal execution?
image


@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Hi @MuddyHope ,

-- Optimized the Dashboard
-- All Checks Passed

Please let me know if there are any issues or further changes needed. Thank you!

@MuddyHope

Copy link
Copy Markdown
Owner

Looks good @Vamsi-aki, thanks for contributing.

@MuddyHope MuddyHope left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM. We will take up the / home page and other enhancements in the future release.

@MuddyHope MuddyHope merged commit 5be4d0b into MuddyHope:main Jul 1, 2025
2 checks passed
@Vamsi-aki

Copy link
Copy Markdown
Contributor Author

Thank you @MuddyHope, for reviewing and merging my first open source PR. I appreciate your patience throughout the process despite the multiple commits. Glad to contribute!

Thank You!

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.

2 participants