Skip to content

Feature/ Sessions fully implemented#44

Merged
ben-AI-cybersec merged 4 commits intoRedback-Operations:mainfrom
RileyTripcony:feature/sessions-implemented
Sep 25, 2025
Merged

Feature/ Sessions fully implemented#44
ben-AI-cybersec merged 4 commits intoRedback-Operations:mainfrom
RileyTripcony:feature/sessions-implemented

Conversation

@RileyTripcony
Copy link
Contributor

Made body_insight table 1:1 relation with activity table. Add error message when trying to create body_insight data when already exists for activity.

Add fields for steps, floors and zone_mintues to activity table.

Also retains previous PR to change dashboard endpoint to get VO2 from database.

Created sleep_data table. Created sleep_data endpoint methods

Created sessions//details to return all information regarding the chosen id. activity data, body_insight data, sleep_data.

… when trying to create a new bodyinsight if already exists for activity. Expanded sessions endpoint with endpoint to get all activity and bodyinsight data by ID.
@github-actions
Copy link

🔒 Security Scan Results

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

Bandit Scan Results:
-------------------
Run started:2025-09-14 11:48:00.119247

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1180
	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.

Copy link

@jd-deakin jd-deakin left a comment

Choose a reason for hiding this comment

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

Change looks okay.
This will be fine in a dev environment.

Fixed user ID's would not be ready for production.
Error handling is good.

@github-actions
Copy link

🔒 Security Scan Results

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

Bandit Scan Results:
-------------------
Run started:2025-09-24 08:00:22.725621

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1365
	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.

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.

lgtm

@ben-AI-cybersec ben-AI-cybersec merged commit 9b7c2dd into Redback-Operations:main Sep 25, 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.

4 participants