docs: Fix token list structure documentation

- Add missing fields: id, network, type, hash
- Remove non-existent logoURI field
- Add field descriptions table
- Link to SDK source for supported networks and types

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: MantisClone

PR_134_Code_Review_Advocate_Kit_SDK_Docs_2026-01-21.md

@@ -0,0 +1,65 @@
+# PR #134 Code Review - Advocate Welcome Kit and SDK Docs Update - 2026-01-21
+
+## PR Summary
+- **Title**: docs: add advocate welcome kit and update SDK quickstart
+- **Author**: AllenAJ
+- **Description**: Added an Advocates Welcome Kit for community onboarding, enhanced the "Updating a Request" guide with functional code examples, updated the Node.js Quickstart with viem integration and request update logic, and updated FAQ and navigation.
+
+## File Groups
+
+### Group 1: SDK Documentation (Code Examples) - High Priority
+- [ ] `docs/advanced/request-network-sdk/get-started/quickstart-node.js.md`
+- [ ] `docs/advanced/request-network-sdk/sdk-guides/request-client/updating-a-request.md`
+**Status**: Not Started
+
+### Group 2: Advocate Program Documentation - Medium Priority
+- [ ] `docs/general/advocates-welcome-kit.md`
+**Status**: Not Started
+
+### Group 3: Navigation Updates - Low Priority
+- [ ] `docs/SUMMARY.md`
+- [ ] `docs/faq.md`
+**Status**: Not Started
+
+## Review Updates
+- **2026-01-21 21:55 UTC**: Posted reply comments for issues #2 and #4
+- **2026-01-21 21:56 UTC**: Pushed fix commit (087dc01) to contributor's branch - replaced viem tab with note, restored addStakeholders row
+- **2026-01-21 22:00 UTC**: Pushed fix commit (26bd317) - changed Discord link to use consistent format (request.network/discord)
+
+## Issues Identified
+
+| Index | File | Issue | Severity | Source | Comment Status | Validation | Resolution | Comment |
+|-------|------|-------|----------|--------|----------------|------------|------------|---------|
+| 1 | quickstart-node.js.md | Viem integration uses `mainnet` chain instead of `sepolia` (inconsistent with rest of quickstart) | High | Greptile | POSTED | VALID | FIXED | Covered by #2 fix - replaced viem tab entirely |
+| 2 | quickstart-node.js.md | Viem `http()` transport is NOT EIP-1193 compatible - Web3Provider conversion will fail at runtime | Critical | CodeRabbit | POSTED | VALID | FIXED | Replaced viem tab with note pointing to ethers v5 |
+| 3 | quickstart-node.js.md | Viem-to-ethers conversion pattern differs from browser quickstart's working implementation | High | Greptile | POSTED | VALID | FIXED | Same as #2 |
+| 4 | updating-a-request.md | Missing `addStakeholders` action from the summary table (was in original) | Low | Greptile | POSTED | VALID | FIXED | Added addStakeholders row back to table |
+
+## Review Progress
+
+### Group 1: SDK Documentation (Code Examples) - Not Started
+**General Description**: TBD
+
+**Files**: quickstart-node.js.md, updating-a-request.md
+
+**Issues Found**: See Issues Identified table (indices 1, 2, 3, 4)
+
+---
+
+### Group 2: Advocate Program Documentation - Not Started
+**General Description**: TBD
+
+**Files**: advocates-welcome-kit.md
+
+**Issues Found**: None identified yet
+
+---
+
+### Group 3: Navigation Updates - Not Started
+**General Description**: TBD
+
+**Files**: SUMMARY.md, faq.md
+
+**Issues Found**: None identified yet
+
+---

docs/general/request-network-token-list.md

@@ -20,16 +20,32 @@ Each token in the list contains the following information:
 
 ```json
 {
-  "id": "TKN-mainnet"
-  "name": "Token Name",
-  "address": "0x...",
-  "symbol": "TKN",
-  "decimals": 18,
-  "chainId": 1,
-  "logoURI": "https://..."
+  "id": "USDC-mainnet",
+  "name": "USD Coin",
+  "symbol": "USDC",
+  "decimals": 6,
+  "address": "0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48",
+  "network": "mainnet",
+  "type": "ERC20",
+  "hash": "0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48",
+  "chainId": 1
 }
 ```
 
+| Field | Description |
+|-------|-------------|
+| `id` | Unique identifier, typically `SYMBOL-network` (e.g., `USDC-mainnet`) |
+| `name` | Human-readable token name |
+| `symbol` | Token symbol |
+| `decimals` | Number of decimal places |
+| `address` | Token contract address |
+| `network` | Network name (e.g., `mainnet`, `matic`, `bsc`) |
+| `type` | Currency type (e.g., `ERC20`, `ETH`, `ISO4217`) |
+| `hash` | For ERC20 tokens, same as `address`. For native tokens, a calculated hash. |
+| `chainId` | Chain ID of the network |
+
+See the [SDK source code](https://github.com/RequestNetwork/requestNetwork/blob/master/packages/types/src/currency-types.ts) for the full list of supported networks and types.
+
 ## Adding a New Token
 
 We welcome community contributions! To add a new token to the list: