Skip to content

feat(APP-517, APP-545): Add Cryptex rewards API integration, slot-based escrow resolution, and XMAQUINA peaq DAO#1028

Merged
thekidnamedkd merged 16 commits intomainfrom
app-517-cryptex-rewards-api-json-generation
Mar 19, 2026
Merged

feat(APP-517, APP-545): Add Cryptex rewards API integration, slot-based escrow resolution, and XMAQUINA peaq DAO#1028
thekidnamedkd merged 16 commits intomainfrom
app-517-cryptex-rewards-api-json-generation

Conversation

@thekidnamedkd
Copy link
Contributor

@thekidnamedkd thekidnamedkd commented Mar 13, 2026

Description

Implements the Cryptex token reward distribution flow (APP-517) and adds the XMAQUINA peaq DAO header (APP-545).

Cryptex rewards API (APP-517):

  • Fix API endpoint URL to match backend (/v2/tokens/rewards/)
  • Hardcode token voting plugin addresses per Cryptex DAO
  • Replace gauge voter plugin lookup with slot-based voting escrow address resolution, making the capital distributor
    plugin-agnostic
  • Add CAPITAL_DISTRIBUTOR_VOTING_ESCROW_ADDRESS slot ID consumed by campaign creation
  • Register escrow addresses for all DAOs via slot functions (Cryptex, Katana, capitalDistributorTest)
  • Add tokenCDTest sepolia DAO under src/daos/cryptex/ for testing
  • Point backend URL to sandbox environment

XMAQUINA peaq DAO (APP-545):

  • Register peaq-mainnet-0x38E4512E4Dbf69F526E30c4F58862bF0C7b26201 with the same XmaquinaPageHeader component
  • Make governance action link dynamic using dao.network and dao.address instead of hardcoded path

DAO registration cleanup:

  • Restructure capitalDistributorTestsrc/daos/katana/, bundling Katana mainnet DAOs with the test DAO
  • Bundle tokenCDTest under src/daos/cryptex/
  • Fix OOM crash caused by barrel import from @/daos in providersDialogs.ts — dialog definitions now imported via
    dedicated daosDialogsDefinitions.ts
  • Add TODO comments referencing APP-558 for test DAO removal

Type of Change

  • Minor: Feature (non-breaking change which adds new functionality)

Developer Checklist:

  • Manually smoke tested the functionality in a preview or locally
  • Confirmed there are no new warnings or errors in the browser console
  • (For User Stories only) Double-checked that all Acceptance Criteria are satisfied
  • Confirmed there are no new warnings on automated tests
  • Merged and published any dependent changes in downstream modules
  • Selected the correct base branch
  • Commented the code in hard-to-understand areas
  • Followed the code style guidelines of this project
  • Reviewed that the Files Changed in Github's UI reflect my intended changes
  • Confirmed the pipeline checks are not failing

Review Checklist:

  • (For User Stories only) Tested in a preview or locally that all Acceptance Criteria are satisfied
  • Confirmed that changes follow the code style guidelines of this project

@linear
Copy link

linear bot commented Mar 13, 2026

@thekidnamedkd thekidnamedkd marked this pull request as ready for review March 13, 2026 18:39
@thekidnamedkd thekidnamedkd requested a review from a team as a code owner March 13, 2026 18:39
@github-actions
Copy link

github-actions bot commented Mar 16, 2026

🚀 Preview Deployment: View Here

@github-actions
Copy link

github-actions bot commented Mar 16, 2026

E2E Smoke Results

Base URL https://app-next-6kgsy5887-aragon-app.vercel.app
Suite smoke
Result ✅ passed
Run #3888

Copy link
Contributor

@milosh86 milosh86 left a comment

Choose a reason for hiding this comment

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

Looks great, left a couple of comments!

@thekidnamedkd thekidnamedkd requested a review from milosh86 March 17, 2026 17:18
Copy link
Contributor

@milosh86 milosh86 left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@thekidnamedkd thekidnamedkd merged commit 2b61eef into main Mar 19, 2026
18 checks passed
@thekidnamedkd thekidnamedkd deleted the app-517-cryptex-rewards-api-json-generation branch March 19, 2026 13:15
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