Skip to content

Resolve issues 487 492 - #592

Merged
Chucks1093 merged 5 commits into
accesslayerorg:mainfrom
dorismaduegbunam:resolve-issues-487-492
Jul 25, 2026
Merged

Resolve issues 487 492#592
Chucks1093 merged 5 commits into
accesslayerorg:mainfrom
dorismaduegbunam:resolve-issues-487-492

Conversation

@dorismaduegbunam

Copy link
Copy Markdown
Contributor

This PR consolidates the resolutions for issues #586, #588, #589, and #590:

1. Unit tests for sell event proceeds matching the sell-path formula output (#586)

2. Integration test for multi-buyer scenario confirming independent holder balances (#588)

3. Structured log event for admin fee bps update (#589)

  • Defines a new structured FeeConfigUpdatedEvent in the contract events module.
  • Emits this event inside set_fee_config with the fields old_bps, new_bps, and updated_at_ledger to support off-chain monitors and indexers.
  • Adds integration tests asserting the event structure and field values.
    closes Add structured log event for admin fee bps update #589

4. Reference docs for the bonding curve formula and off-chain price prediction (#590)

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

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

@dorismaduegbunam
dorismaduegbunam force-pushed the resolve-issues-487-492 branch from d79d4af to eaf0727 Compare July 25, 2026 11:38
@Chucks1093
Chucks1093 merged commit e9fd8bc into accesslayerorg:main Jul 25, 2026
1 check 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