Conversation
| <Aside | ||
| type="note" | ||
| title="Wallet V5 and `wallet_id`" | ||
| > | ||
| Wallet V5 replaces `subwallet_id` with a derived 32‑bit `wallet_id` field that encodes network, wallet version, workchain, and subwallet number in a single value. The on-chain getter is still named `get_subwallet_id()` for backward compatibility, but for Wallet V5 it returns this derived `wallet_id`. See the [Wallet V5 overview](/standard/wallets/v5#wallet-id-scheme) for the exact scheme. |
There was a problem hiding this comment.
[HIGH] Stub “See …” sentence in Wallet V5 aside
The Wallet V5 aside ends with a standalone sentence, “See the Wallet V5 overview for the exact scheme.” This is a “stub” sentence whose sole purpose is to tell the reader to “See …” a link, which the style guide explicitly forbids in running prose. The guideline requires integrating the link naturally into the sentence instead of wrapping it in meta text like “See …”. Rephrasing the sentence keeps the content intact while aligning with the documented link-text and stub-sentence style rule.
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
closes #1342