Skip to content

update dashboard to get vo2 data from database. update requirements.txt#37

Merged
ben-AI-cybersec merged 1 commit intoRedback-Operations:mainfrom
RileyTripcony:feature/v02-dashboard
Sep 17, 2025
Merged

update dashboard to get vo2 data from database. update requirements.txt#37
ben-AI-cybersec merged 1 commit intoRedback-Operations:mainfrom
RileyTripcony:feature/v02-dashboard

Conversation

@RileyTripcony
Copy link
Contributor

Rebased and merged conflict in .gitignore to keep *.db. Also added .vscode to ignore.

Changed dashboard to get V02 Max from database table Body_insight. It previously had palceholder value of 45.

In testing frontend I was getting V02 Max from body_insight endpoint as well as getting name and profile avatar from dashboard endpoint.

I believe getting all required data from dashboard endpoint is better practice.

Removed comments from last trimester that would be confusing for this trimester.

@github-actions
Copy link

🔒 Security Scan Results

🔒 Security Scan Results
=========================

Bandit Scan Results:
-------------------
Run started:2025-08-24 05:26:11.700157

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1059
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 14
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 14
Files skipped (0):

Dependency Check Results:
-----------------------

No critical security issues detected.

The code has passed all critical security checks.

@Narendran99
Copy link
Contributor

Hi Riley.
The changes to the Dashboard.py are good. But the requirements.txt file seems to be empty. If there are any elements to be added to the file, add those and I can merge the file.

@RileyTripcony RileyTripcony force-pushed the feature/v02-dashboard branch from 2a9a4e2 to 660c7c6 Compare August 28, 2025 16:04
@github-actions
Copy link

🔒 Security Scan Results

🔒 Security Scan Results
=========================

Bandit Scan Results:
-------------------
Run started:2025-08-28 16:04:29.621601

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1059
	Total lines skipped (#nosec): 0
	Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
	Total issues (by severity):
		Undefined: 0
		Low: 14
		Medium: 0
		High: 0
	Total issues (by confidence):
		Undefined: 0
		Low: 0
		Medium: 0
		High: 14
Files skipped (0):

Dependency Check Results:
-----------------------

No critical security issues detected.

The code has passed all critical security checks.

@RileyTripcony
Copy link
Contributor Author

Thank you for reviewing. I am unsure as to why it was empty. I have removed it from this pull request. I will create a seperate PR in future with updated requirements.txt.

Copy link
Member

@ben-AI-cybersec ben-AI-cybersec left a comment

Choose a reason for hiding this comment

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

This is fine, thanks

@ben-AI-cybersec ben-AI-cybersec merged commit cc4d057 into Redback-Operations:main Sep 17, 2025
1 check passed
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