Skip to content

Issue: 89 90 91 92#155

Open
blessedcodey-boy wants to merge 4 commits into
Tx-wat:mainfrom
blessedcodey-boy:89-90-91-92
Open

Issue: 89 90 91 92#155
blessedcodey-boy wants to merge 4 commits into
Tx-wat:mainfrom
blessedcodey-boy:89-90-91-92

Conversation

@blessedcodey-boy
Copy link
Copy Markdown

Summary

Implement four contract detail page enhancements to improve UX and information clarity.

Changes

Issue #89: Add direct open link for webhook URL

  • Convert webhook URL from plain text to clickable link
  • Opens in new tab with target="_blank" and rel="noopener noreferrer"
  • Uses indigo-400 hover styling consistent with other links on the page
  • Allows users to quickly test webhook endpoints

Issue #90: Add contract creation time with locale-friendly formatting

  • Display precise creation timestamp alongside date
  • Shows both date and time using formatDate() and toLocaleTimeString()
  • Automatically adapts to user's browser locale settings
  • Improves ability to identify when contracts were registered

Issue #91: Add "last alert" summary to contract detail header

  • Replace static "Total Alerts" card with dynamic content
  • Show "Last Alert" with timestamp when alerts exist
  • Display "Total Alerts: No alerts yet" when no alerts present
  • Include both date and time for latest alert
  • Enables quick scanning of contract activity status

Issue #92: Improve delete confirmation copy and scope

  • Enhance delete modal with explicit, detailed messaging
  • Add bulleted list clarifying exactly what will be deleted:
    • Contract configuration and alert rules
    • All alert records (with dynamic count)
  • Make deletion scope unambiguous and prevent accidental data loss
  • Improve UX by setting clear expectations

Technical Details

File Modified: app/contracts/[id]/page.tsx

Changes:

  • Webhook URL: Changed from <p> to <a> tag with proper link styling
  • Registration timestamp: Added toLocaleTimeString() display below date
  • Last alert card: Conditional rendering based on alerts array length
  • Delete modal: Restructured with improved copy and bulleted list of deleted items

No Breaking Changes: All modifications are additive and maintain backward compatibility.

Testing

  • Verify webhook URL is clickable and opens in new tab
  • Confirm timestamps display in user's locale format
  • Check last alert shows correctly when alerts exist and "No alerts yet" when empty
  • Test delete modal displays accurate alert count and clear messaging

Closes #89
Closes #90
Closes #91
Closes #92

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 30, 2026

@blessedcodey-boy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant