Unused variables on main hide logic that was lost in merges (previousStatus, existingPool, callerUserId…)

Đang mở
  • Có gì không ổn?
#605 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@urielit1313-creator đang làm issue này rồi.

Từ ngày 13/10/2026.

  • #631 của @urielit1313-creator — đang mở
  • #656 của @yggdrasilworld — đã đóng, không merge

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
25/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
eslint, typescript
Lĩnh vực
backend

Hướng nghiên cứu

Start with the open pull requests #631 and #656, since two linked PRs already target this cleanup. The unused bindings are in bounties.service.ts (previousStatus in markMergedAndRelease, callerUserId in create), maintenance-pool.service.ts (existingPool and existingPayment), repo-scope.guard.ts (ROLES_KEY) and bounties.controller.ts (ClaimBountyDto). Done means eslint no-unused-vars passes, and the previousStatus rollback question is answered explicitly, either restored or removed on purpose.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug good first issue

Problem

eslint reports no-unused-vars errors at:

  • bounties.service.ts:165: previousStatus assigned in markMergedAndRelease and never used, which suggests the status rollback on release failure was dropped.
  • bounties.service.ts:38: callerUserId param of create ignored, so the sponsor identity comes from the DTO (#40).
  • maintenance-pool.service.ts:88: existingPool fetched and ignored in the race-loser branch.
  • maintenance-pool.service.ts:147: existingPayment (dead lookup).
  • repo-scope.guard.ts:16: ROLES_KEY.
  • bounties.controller.ts:22 and its spec: ClaimBountyDto.

Suggested fix

Remove or reconnect each one. In particular, confirm whether previousStatus was supposed to restore the bounty on a failed release.

Related: #46, #259

Ngôn ngữ chính
TypeScript
Star
2
Fork
90
Merge trung bình
31 phút
Pull request đã merge (30 ngày)
20

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của MergeFi/backend

Tất cả issue của MergeFi/backend

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.