Skip to content

fix: resolve issues - #242

Merged
nonsobethel0-dev merged 1 commit into
Parashield-Protocol:mainfrom
edehvictor:codex-211-213-quickfix
Jul 25, 2026
Merged

nonsobethel0-dev merged 1 commit into
Parashield-Protocol:mainfrom
edehvictor:codex-211-213-quickfix

Conversation

@edehvictor

Copy link
Copy Markdown
Contributor

Closes #211
Closes #213
Closes #210
Closes #212

Summary

This PR fixes the two quickest assigned regressions from the current issue batch:

Changes

  • Updated shortenAddress() to use symmetric truncation (4…4 by default).
  • Extracted getPolicyTimelineEvents() so timeline state is easier to verify.
  • Marked the Oracle monitoring step as complete for Active, Claimed, and Expired policies.
  • Added regression tests for both fixes.

Issue Tracking

Testing

  • Added unit coverage for symmetric address truncation.
  • Added unit coverage for claimed/expired timeline completion behavior.
  • Full local test run was not completed here because vitest is not installed in this checkout.

@netlify

netlify Bot commented Jul 25, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for boisterous-sunshine-dd4c4c failed.

Name Link
🔨 Latest commit c7adab8
🔍 Latest deploy log https://app.netlify.com/projects/boisterous-sunshine-dd4c4c/deploys/6a64af4bf492580008dbf36e

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@edehvictor 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

@nonsobethel0-dev
nonsobethel0-dev merged commit 288a071 into Parashield-Protocol:main Jul 25, 2026
0 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment