EasyTier / EasyTier

A simple, decentralized mesh VPN with WireGuard support.
https://easytier.top
Apache License 2.0
459 stars 41 forks source link

Perf/ts type #100

Closed m1m1sha closed 1 month ago

m1m1sha commented 1 month ago

是否需要通过 serde rename_all 将 rs snake_case 转换为 camelCase以符合 ts 命名规则 可能会导致 ts 端按 snake_case 命名规则储存的数据无法解析,需要做兼容处理

KKRainbow commented 1 month ago

是否需要通过 serde rename_all 将 rs snake_case 转换为 camelCase以符合 ts 命名规则 可能会导致 ts 端按 snake_case 命名规则储存的数据无法解析,需要做兼容处理

你说的兼容性问题是指从 LocalStorage 里读出的数据可能会不兼容么

m1m1sha commented 1 month ago

是否需要通过 serde rename_all 将 rs snake_case 转换为 camelCase以符合 ts 命名规则 可能会导致 ts 端按 snake_case 命名规则储存的数据无法解析,需要做兼容处理

你说的兼容性问题是指从 LocalStorage 里读出的数据可能会不兼容么

对,因为之前储存的 key 是 snake_case ,但是数据储存在 localStorage 的话,版本更新还能读取到吗 是不是应该专门存配置文件

KKRainbow commented 1 month ago

是否需要通过 serde rename_all 将 rs snake_case 转换为 camelCase以符合 ts 命名规则 可能会导致 ts 端按 snake_case 命名规则储存的数据无法解析,需要做兼容处理

你说的兼容性问题是指从 LocalStorage 里读出的数据可能会不兼容么

对,因为之前储存的 key 是 snake_case ,但是数据储存在 localStorage 的话,版本更新还能读取到吗 是不是应该专门存配置文件

我感觉不用考虑兼容性了吧,如果无法解析 localstorage,就把之前的清理掉好了

m1m1sha commented 1 month ago

是否需要通过 serde rename_all 将 rs snake_case 转换为 camelCase以符合 ts 命名规则 可能会导致 ts 端按 snake_case 命名规则储存的数据无法解析,需要做兼容处理

你说的兼容性问题是指从 LocalStorage 里读出的数据可能会不兼容么

对,因为之前储存的 key 是 snake_case ,但是数据储存在 localStorage 的话,版本更新还能读取到吗 是不是应该专门存配置文件

我感觉不用考虑兼容性了吧,如果无法解析 localstorage,就把之前的清理掉好了

嗯嗯,命名风格先不变了,修改的话修改不彻底,有些 prost 生成的结构体好像无法用 rename