Skip to content

feat: standardize token event schemas and add Makefile preflight target#139

Merged
BigBen-7 merged 1 commit intoLead-Studios:mainfrom
Cedarich:feat/standardize-event-schemas-and-add-preflight-check
Mar 30, 2026
Merged

feat: standardize token event schemas and add Makefile preflight target#139
BigBen-7 merged 1 commit intoLead-Studios:mainfrom
Cedarich:feat/standardize-event-schemas-and-add-preflight-check

Conversation

@Cedarich
Copy link
Copy Markdown

Summary

This PR implements two key improvements to the Veritix Pay contract:

  1. Standardized Token Event Schemas - All core token operations now follow consistent event topic/payload conventions
  2. Makefile Preflight Target - Fast validation of required developer tooling

Changes

Event Schema Standardization

Standardized event schemas across all token operations to improve indexer and downstream integration maintainability:

Core Token Events

  • mint: (mint, admin, to)amount
  • burn: (burn, from)amount
  • burn_from: (burn, spender, from)amount
  • transfer: (transfer, from, to)amount
  • approve: (approve, from, spender)amount
  • clawback: (clawback, admin, from)amount

Module Events

  • escrow_created/released/refunded: Consistent (event_name, id, address) format
  • dispute_opened/resolved: Simplified to single symbol identifiers
  • recurring_setup/executed/cancelled: Standardized payload structure
  • split_distributed: Single symbol with structured payload

All events now use symbol_short!() for the primary event identifier, following Soroban best practices and reducing gas costs.

Developer Tooling - Preflight Target

Added make preflight target to validate required tools before build:

make preflight  # verify all required tools are installed

Closes #100 
Closes #99 

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 29, 2026

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

@BigBen-7 BigBen-7 merged commit 18acd33 into Lead-Studios:main Mar 30, 2026
1 check failed
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.

3 participants