Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
synapse-dev | cc40389 | Commit Preview URL Branch Preview URL |
Feb 24 2026, 01:25 PM |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
snissn
left a comment
There was a problem hiding this comment.
Deep-dive review complete. I found and fixed a set of docs API-drift issues in follow-up PR #629 (https://github.com/FilOzone/synapse-sdk/pull/629):\n\n- React guide examples now handle optional wallet address correctly under strict TS settings\n- Storage Context guide examples updated to current options-object download API and valid upload input types\n- Synapse and Synapse Core signatures/examples corrected for current SDK behavior\n\nMergeability status: #628 is CLEAN against master.\n\nRecommendation: once #629 is incorporated into this branch/work (via merge/cherry-pick/rebase), I approve this docs refresh for merge. Without those fixes, I recommend holding merge due to misleading example usage in key developer guides.
Refreshes developer guide documentation to reflect all synapse packages.
@filoz/synapse-reacthooks@filoz/synapse-coremodules usage examples