-
Notifications
You must be signed in to change notification settings - Fork 86
chore: ctoken extensions cleanup #2161
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
Changes from all commits
4158fb7
c17726b
d567e6e
1917f52
c4dff36
9db54e8
54a4461
c889069
bf219fb
c72b0a9
13965d5
e3b7bdb
ff1f49b
4abf946
4e4a709
13d992b
8c2e9f1
4f2c674
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,8 +7,8 @@ use crate::{ | |
|
|
||
| /// Calculates the size of a ctoken account based on which extensions are present. | ||
| /// | ||
| /// Note: Compression info is now embedded in the base struct (CTokenZeroCopyMeta), | ||
| /// so there's no separate compressible extension parameter. | ||
| /// Note: Compressible extension is required if the T22 mint has restricted extensions | ||
| /// (Pausable, PermanentDelegate, TransferFee, TransferHook). | ||
|
Comment on lines
+10
to
+11
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Include DefaultAccountState in the list of restricted extensions. Your update correctly documents that the compressible extension is required for mints with restricted extensions, but the list still omits DefaultAccountState. The DefaultAccountStateExtension sets a default state for all new token accounts created for a mint. The validation logic (per the prior review) enforces this across five restricted extensions: Pausable, PermanentDelegate, TransferFeeConfig, DefaultAccountState, and TransferHook. The comment should match the implementation. 🔎 Proposed fix/// Note: Compressible extension is required if the T22 mint has restricted extensions
-/// (Pausable, PermanentDelegate, TransferFee, TransferHook).
+/// (Pausable, PermanentDelegate, DefaultAccountState, TransferFee, TransferHook).🤖 Prompt for AI Agents |
||
| /// | ||
| /// # Arguments | ||
| /// * `extensions` - Optional slice of extension configs | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.