Skip to content

fix: UX Improvements Audit Fixes #554

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.

Sign up for GitHub

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

Jump to bottom
Open
ypatil12 wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

fix: UX Improvements Audit Fixes #554

ypatil12 wants to merge 4 commits into from

Conversation

@ypatil12
Copy link
Collaborator

@ypatil12 ypatil12 commented Jan 6, 2026 •
edited
Loading

Motivation:

Audit fixes for all informational issues.

Modifications:

  • I-04: Adds the slasher address in the QuorumCreated event
  • I-01: Add clarifying documentation around calling of getNonSignerWitnessesAndApk
  • I-02: Add clarifying documentation around offchain usage of getNonSignerWitnessesAndApk
  • I-03: Add clarifying documentation getOperatorInfos not being used to derive operatorIndex

Result:

Audit fixes ready

@ypatil12 ypatil12 force-pushed the feat/ux-updates-audit-fixes branch 3 times, most recently from df8270a to 7e56422 Compare January 6, 2026 18:21
@ypatil12 ypatil12 force-pushed the feat/ux-updates-audit-fixes branch from 7e56422 to 875d6cb Compare January 8, 2026 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ypatil12