Skip to content

fix: correct details sidebar labels and values for propose/verify/settled pages#265

Open
pumpedlunch wants to merge 1 commit intomasterfrom
fix/details-sidebar-labels
Open

fix: correct details sidebar labels and values for propose/verify/settled pages#265
pumpedlunch wants to merge 1 commit intomasterfrom
fix/details-sidebar-labels

Conversation

@pumpedlunch
Copy link
Collaborator

@pumpedlunch pumpedlunch commented Mar 20, 2026

Summary

  • Show page-aware labels in the details sidebar: "Challenge period length" on propose, "Minimum challenge period ends" for Managed OO on verify, "Challenge period ends" otherwise
  • Show "Proposer Reward" / "Disputer Reward" with correct values per page instead of a generic "Reward" label
  • Hide "Proposal: " prefix on the settled page for single and multiple-value requests

Files changed

  • libs/src/oracle-sdk-v2/services/oraclev2/ethers/factory.ts — forward customLiveness from on-chain contract read into SharedRequest
  • src/types/ui.ts — add customBond field to OracleQueryUI
  • src/helpers/converters.ts — populate customBond in requestToOracleQuery and assertionToOracleQuery
  • src/components/Panel/Actions/Details.tsx — page-aware labels, formatLivenessDuration, disputer reward display
  • src/components/Panel/Actions/Actions.tsx — hide "Proposal: " prefix on settled page

Test plan

  • Navigate to a propose URL with a MOOV2 request that has customLiveness = 3600 — verify "Challenge period length" shows "60 minutes" (not "2 hours")
  • Verify page shows "Disputer Reward" with correct value (bond / 2), not the proposer reward
  • Settled page shows the resolved value without "Proposal: " prefix
  • Propose page shows "Proposer Reward" and "Challenge period length" labels

🤖 Generated with Claude Code

…tled pages

- Show page-aware labels in the details sidebar: "Challenge period length"
  on propose, "Minimum challenge period ends" for managed OO on verify,
  "Challenge period ends" otherwise
- Show "Proposer Reward" / "Disputer Reward" labels with correct values
  per page instead of a generic "Reward" label
- Hide "Proposal: " prefix on the settled page for single and multiple value requests
- Fix challenge period showing "2 hours" instead of actual custom liveness:
  `customLiveness` was fetched from the contract via `fetchRequest()` but
  never forwarded in `ConvertToSharedRequest` in the OOV2 ethers factory
- Add `customBond` to `OracleQueryUI` to hold the raw requester-set bond
  before finalFee is merged in, used to compute disputer reward (bond / 2)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
amoy-optimistic-oracle-dapp-v2 Ready Ready Preview, Comment Mar 20, 2026 5:41pm
optimistic-oracle-dapp-v2 Ready Ready Preview, Comment Mar 20, 2026 5:41pm
sepolia-optimistic-oracle-dapp-v2 Ready Ready Preview, Comment Mar 20, 2026 5:41pm

Request Review

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.

1 participant