Skip to content

chore(catalog): show Desktop users UpRock's real 250 UPT minimum (mirror of GeiserX/CashPilot-Desktop#264) - #21

Closed
geiresx wants to merge 1 commit into
mainfrom
ws-catalog-sync-uprock
Closed

geiresx wants to merge 1 commit into
mainfrom
ws-catalog-sync-uprock

Conversation

@geiresx

@geiresx geiresx commented Nov 7, 2026 •

Copy link
Copy Markdown
Owner

Review mirror; findings are fixed on the real PR.

Summary by CodeRabbit

  • Documentation
    • Updated payout details for several services, including payout estimates, minimums, supported methods, and withdrawal requirements.
    • Clarified that Honeygain balances are deducted when a payout is requested, even while processing is pending.
    • Updated Proxybase.xyz information to state that USDC payouts can be sent to any EVM address controlled by the recipient.
    • Repocket’s listed payment methods now include gift cards and crypto, with PayPal removed.
    • Uprock’s minimum withdrawal is listed as 250 UPT, with Solana destination, two-factor authentication, and fees noted.

… fixes

Vendors services/ from the web branch fix/uprock-minimum-payout
(GeiserX/CashPilot#641): UpRock's minimum becomes 250 UPT. The sync also
brings the web catalog changes already on web main that Desktop had not
picked up yet (earnapp, honeygain, proxybase, proxybase-xyz, repocket).
@coderabbitai

coderabbitai Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

Explain Change →

Walkthrough

Six service profiles now contain updated payout methods, minimums, destination requirements, processing details, and payout records.

Changes: Service payout profiles

Layer / File(s) Summary
Payment options and minimums
services/bandwidth/repocket.yml, services/depin/uprock.yml
Repocket’s payment methods now list gift cards and crypto, with USDT configured as the crypto token. Uprock’s withdrawal minimum changes from USD 5 to 250 UPT.
Payout destinations and processing
services/bandwidth/proxybase-xyz.yml, services/bandwidth/honeygain.yml, services/depin/uprock.yml
Proxybase-xyz’s payout terms now describe withdrawals to EVM addresses whose keys the user holds. Honeygain’s notes add Tipalti processing, SMS verification, and tax-form details. Uprock’s notes describe Solana withdrawals, two-factor authentication, and a fee.
Payout records and outcomes
services/bandwidth/earnapp.yml, services/bandwidth/proxybase.yml, services/bandwidth/repocket.yml
EarnApp’s October 2026 record describes a payment about 2% below the redeemed balance. Proxybase’s record describes a withdrawal under $1. Repocket’s notes report a refused Tremendous request and a successful USDT-on-Polygon request.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: Minimal · up to ae252

The payout profiles reflect updated withdrawal terms. No material merge-blocking risk has been established.

Architecture risk: Low · up to ae252

The change affects 2 systems.

Changed systems: services/bandwidth, services/depin

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — services/bandwidth (service) was modified; 5 changed files map to changed impact.
  • observed — services/depin (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in services/bandwidth/earnapp.yml: The cashout note replaces the specific $37.54 balance and $36.81 PayPal receipt with a general statement that an October 2026 payment arrived about 2% below the redeemed balance.
  • observed — Modified behavior in services/bandwidth/earnapp.yml: The payout-record pro removes the specific balance and receipt amounts while retaining the October 2026 date and approximate 2% shortfall.
  • observed — Modified behavior in services/bandwidth/honeygain.yml: The cashout notes add that the balance is deducted upon request while Tipalti processing may still be pending, later logins require a new SMS code unless the device is remembered for 30 days, and a payee in Spain was not asked for a tax form in October 2026.
  • observed — Modified behavior in services/bandwidth/proxybase-xyz.yml: The description expands MPP from “Micropayments Protocol” to “Machine Payments Protocol.”
Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title identifies the main change: updating UpRock's minimum payout to 250 UPT. The other catalog changes came from the sync.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (6 skipped: 6 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 6
  • Waived: $1.50 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Additional comments: 1

services/bandwidth/proxybase-xyz.yml-75-75: 1

75-75: Minor · Functional Correctness · Quick win

Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.

Explain how to display the payout token.

Adding Tempo may not make the token visible. MetaMask does not list Tempo among its enhanced token-detection networks and may require a manual token import. (support.metamask.io)

Tell users to import the payout token if it does not appear.

Review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 67ddd59 and ae25208.

Files selected for processing: 6

  • services/bandwidth/earnapp.yml
  • services/bandwidth/honeygain.yml
  • services/bandwidth/proxybase-xyz.yml
  • services/bandwidth/proxybase.yml
  • services/bandwidth/repocket.yml
  • services/depin/uprock.yml

Limit details: You’ve used all 10 included reviews currently available.

@geiresx

geiresx commented Nov 7, 2026

Copy link
Copy Markdown
Owner Author

No findings; review mirror closed.

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