docs: fix protocol claim fees account list - #228
Conversation
Update the `process_protocol_claim_fees` account documentation to match the accounts required by the implementation. The docs were missing the `delegation_program_data` account, while the code requires it to load the protocol program upgrade authority. The account list now documents all three accounts with their correct indices and order.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughUpdated the protocol fee processor account documentation. The documentation now identifies the admin signer and protocol fees vault by index and adds the delegation program data account at index 2. Merge Risk: ⚪ Minimal · up to This documentation-only update adds the required account entry and does not introduce an actionable merge-blocking risk; it is merge-ready after normal checks and review. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Problem
The
process_protocol_claim_feesdocumentation was missing thedelegation_program_dataaccount required by the implementation.Solution
Updated the account documentation to include all three required accounts with their correct indices and order.
The
delegation_program_dataaccount is required by the code to load the protocol program upgrade authority.Before & After Screenshots
Documentation-only change.
Deploy Notes
No deployment changes.
Summary by CodeRabbit