Skip to content

fix: validate network and document state edge cases - #1242

Merged
mftee merged 1 commit into
CodeGirlsInc:mainfrom
Mkalbani:feat/issue-1205-1204-1202-1203-network-chain-revoke-transfer
Aug 26, 2026
Merged

fix: validate network and document state edge cases#1242
mftee merged 1 commit into
CodeGirlsInc:mainfrom
Mkalbani:feat/issue-1205-1204-1202-1203-network-chain-revoke-transfer

Conversation

@Mkalbani

Copy link
Copy Markdown
Contributor

Summary

Fixes edge-case validation for network configuration, ownership chains, revocation, and transfers.

Changes

Testing

  • Added focused Rust tests for all four cases.
  • cargo fmt --check
  • cargo test

Issues

Closes #1205
Closes #1204
Closes #1202
Closes #1203

@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

@Mkalbani is attempting to deploy a commit to the Mftee's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

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

@mftee mftee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No merge conflicts with main. Good handling of the network and document state edge cases - approving.

@mftee
mftee merged commit e1b29d7 into CodeGirlsInc:main Aug 26, 2026
0 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants