Skip to content

background passive mining#2616

Merged
J43fura merged 43 commits intomainfrom
1863-background-passive-mining-2
Feb 25, 2026
Merged

background passive mining#2616
J43fura merged 43 commits intomainfrom
1863-background-passive-mining-2

Conversation

@J43fura
Copy link
Collaborator

@J43fura J43fura commented Feb 23, 2026

No description provided.

@J43fura J43fura requested a review from malek10xdev February 23, 2026 11:58
@J43fura J43fura self-assigned this Feb 23, 2026
@J43fura J43fura linked an issue Feb 23, 2026 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Feb 23, 2026

Coverage Report

Passed

Commit: 3e88b7f

Summary

Name Stmts Branch Funcs Lines
🔴 Total 37% 33.9% 42.9% 37.1%
All files
Name Stmts Branch Funcs Lines
🔴 backend/src/app.ts 0% 0% 0% 0%
🔴 backend/src/server.ts 0% 100% 0% 0%
🔴 backend/src/controllers/mining.controller.ts 0% 0% 0% 0%
🔴 backend/src/controllers/stream.controller.ts 0% 0% 0% 0%
🔴 backend/src/middleware/auth.ts 0% 0% 0% 0%
🔴 backend/src/routes/mining.routes.ts 0% 100% 0% 0%
🔴 backend/src/routes/stream.routes.ts 0% 100% 0% 0%
🟡 backend/src/services/tasks-manager/TasksManager.ts 54.3% 27.6% 77.8% 55.5%
🔴 backend/src/services/tasks-manager/TasksManagerFile.ts 0% 0% 0% 0%
No coverage changes
Name Stmts Branch Funcs Lines
🔴 backend/src/app.ts 0% 0% 0% 0%
🔴 backend/src/server.ts 0% 100% 0% 0%
🔴 backend/src/controllers/mining.controller.ts 0% 0% 0% 0%
🔴 backend/src/controllers/stream.controller.ts 0% 0% 0% 0%
🔴 backend/src/middleware/auth.ts 0% 0% 0% 0%
🔴 backend/src/routes/mining.routes.ts 0% 100% 0% 0%
🔴 backend/src/routes/stream.routes.ts 0% 100% 0% 0%
🟡 backend/src/services/tasks-manager/TasksManager.ts 54.3% 27.6% 77.8% 55.5%
🔴 backend/src/services/tasks-manager/TasksManagerFile.ts 0% 0% 0% 0%

Generated by Test Coverage Reporter for commit 3e88b7f

Copy link
Collaborator

@malek10xdev malek10xdev left a comment

Choose a reason for hiding this comment

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

Can't find the changes where passive_mining will be passed to tasks-manager.crateTask, is it implemented or still ?


if (userId) {
// Create a service user object with the extracted userId
const serviceUser: User = {
Copy link
Collaborator

@malek10xdev malek10xdev Feb 24, 2026

Choose a reason for hiding this comment

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

Use supabase.getUser to validate that id actually exists and also you have complete user object instead of hardcoded

// Extract userId from route or query params
const userId = req.params.userId ?? req.query.userId;

if (userId) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Fetch user and wait for response if userId doesn't exist return status 404 if error 500 otherwise next()

@J43fura J43fura requested a review from malek10xdev February 24, 2026 20:05
@J43fura J43fura merged commit 5e29d93 into main Feb 25, 2026
14 of 15 checks passed
@J43fura J43fura deleted the 1863-background-passive-mining-2 branch February 25, 2026 12:41
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.

Background passive mining

2 participants