Repository navigation
Conversation
STORAGE_TYPE_S3, STORAGE_TYPE_DATABASE, STORAGE_TYPE_LOCAL are the required protojson enum names for storages[].type. The short names (S3, DATABASE, LOCAL) fail startup with an invalid enum value error. Fixes usememos/memos#6407
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe deployment configuration guide now lists Suggested reviewers: Priority: ➖ Normal Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to The deployment guide now uses storage type values accepted by the configuration parser. No actionable merge risk is evident from this documentation change. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
Fixes usememos/memos#6407
The deployment-managed storage example on the docs site used short protojson enum names (
"type": "S3", and prose listingDATABASE/LOCAL/S3). Those fail to decode: storages[].type expectsSTORAGE_TYPE_S3,STORAGE_TYPE_DATABASE, andSTORAGE_TYPE_LOCAL.This updates
content/docs/configuration/deployment-configuration.mdxonly so the published example matches what Memos 0.31 accepts. No application code changes (decode-error improvement is tracked separately in usememos/memos#6415).