Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor events to improve parameter order and indexing (#787) #817

Merged
merged 35 commits into from
Dec 17, 2024

Conversation

doerfli
Copy link
Contributor

@doerfli doerfli commented Dec 10, 2024

@doerfli doerfli force-pushed the feature/events-refactoring branch from 7ef5c72 to 185308b Compare December 10, 2024 16:13
@doerfli doerfli linked an issue Dec 12, 2024 that may be closed by this pull request
4 tasks
contracts/distribution/IDistributionService.sol Outdated Show resolved Hide resolved
contracts/instance/IInstance.sol Outdated Show resolved Hide resolved
contracts/instance/base/BalanceStore.sol Outdated Show resolved Hide resolved
contracts/pool/IBundleService.sol Outdated Show resolved Hide resolved
contracts/pool/IPoolService.sol Outdated Show resolved Hide resolved
contracts/product/IPolicyService.sol Outdated Show resolved Hide resolved
contracts/shared/IComponentService.sol Outdated Show resolved Hide resolved
contracts/staking/IStaking.sol Outdated Show resolved Hide resolved
contracts/staking/IStakingService.sol Outdated Show resolved Hide resolved
contracts/staking/TargetHandler.sol Outdated Show resolved Hide resolved
…indexing for enhanced event filtering and logging (#787)
…parameter indexing for enhanced event filtering (#787)
…e parameter indexing for enhanced event filtering (#787)
…rameter indexing for enhanced event filtering (#787)
@doerfli doerfli force-pushed the feature/events-refactoring branch from d17e379 to 8c8e108 Compare December 16, 2024 21:41
Copy link
Contributor

Todo Commenter

contracts/instance/InstanceService.sol
TODO:
  • rename exception Line: 386
contracts/pool/PoolService.sol
TODO:
contracts/product/ClaimService.sol
TODO:
  • move to policy helper lib or something Line: 608

@doerfli Please take a look 👀

@doerfli doerfli merged commit 4a8758c into develop Dec 17, 2024
12 checks passed
@doerfli doerfli deleted the feature/events-refactoring branch December 17, 2024 08:27
@github-actions github-actions bot locked and limited conversation to collaborators Dec 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve logging and use indexed logs
2 participants