Skip to content

feat: morpho v2 alert tweaks (removeAdapter param + executable countdown)#294

Merged
spalen0 merged 3 commits into
mainfrom
morpho-ms
Jun 26, 2026
Merged

feat: morpho v2 alert tweaks (removeAdapter param + executable countdown)#294
spalen0 merged 3 commits into
mainfrom
morpho-ms

Conversation

@spalen0

@spalen0 spalen0 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Two tweaks to the Morpho V2 governance pending-config alert:

  • removeAdapter param removed — the alert now renders the operation as removeAdapter() instead of removeAdapter(adapter 0x…).
  • Countdown on "Executable at" — appends a human-friendly time-from-now in parentheses, e.g. ⏰ Executable at: 2026-07-02 16:50:35 (3 days). Falls back to hours/minutes when under a day, and (now) once elapsed.

Before

📥 Submitted: removeAdapter(adapter 0x80126555b170957dfed67a3BFbB7893e20fE4fC0)
⏰ Executable at: 2026-07-02 16:50:35

After

📥 Submitted: removeAdapter()
⏰ Executable at: 2026-07-02 16:50:35 (3 days)

Changes

  • protocols/morpho/v2_decoders.py — split addAdapter/removeAdapter so removeAdapter renders with no args.
  • protocols/morpho/governance_v2.py — add _format_countdown helper, append it to the "Executable at" line.
  • tests/test_morpho_v2_decoders.py — update removeAdapter expectation.

Testing

  • pytest tests/test_morpho_v2_decoders.py — 25 passed
  • ruff check — clean

🤖 Generated with Claude Code

codex and others added 3 commits June 26, 2026 08:25
- removeAdapter no longer shows the adapter param: renders as `removeAdapter()`
- "Executable at" line now appends a countdown from now, e.g. (3 days)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@spalen0 spalen0 marked this pull request as ready for review June 26, 2026 10:20
@spalen0 spalen0 merged commit 0fabf4f into main Jun 26, 2026
3 checks passed
@spalen0 spalen0 deleted the morpho-ms branch June 26, 2026 10:21
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