vuejs / pinia

🍍 Intuitive, type safe, light and flexible Store for Vue using the composition api with DevTools support
https://pinia.vuejs.org
MIT License
12.91k stars 1.02k forks source link

docs(zh,en): Update translation of core-concepts/index, move SSR link forward to new mention #2530

Closed PrainLopez closed 6 months ago

PrainLopez commented 8 months ago

packages/docs/zh/core-concepts/index.md 更新翻译同步英文文档现版本,个人经验不足,72行需要一些review Update the docs/zh translation to latest. May need review at line 72 due to my experience.

packages/docs/core-concepts/index.md In the process of translating I found a bug, the line 71 was added after line 73 while the SSR link is remained. Has moved it forward and cancelled the latter one on line 73.

netlify[bot] commented 8 months ago

Deploy Preview for pinia-playground canceled.

Name Link
Latest commit b869f3e3dd52e4aeb940fb5fc992cf4b3626c926
Latest deploy log https://app.netlify.com/sites/pinia-playground/deploys/65a9d311d04f6f0009f51436
netlify[bot] commented 8 months ago

Deploy Preview for pinia-official ready!

Name Link
Latest commit b869f3e3dd52e4aeb940fb5fc992cf4b3626c926
Latest deploy log https://app.netlify.com/sites/pinia-official/deploys/65a9d3117b923e000883335e
Deploy Preview https://deploy-preview-2530--pinia-official.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

PrainLopez commented 7 months ago

Screenshot 2024-01-19 092350

packages/docs/core-concepts/index.md In the process of translating I found a bug, the line 71 was added after line 73 while the SSR link is remained. Has moved it >forward and cancelled the latter one on line 73.

This is now causing conflict, should I submit it in a seperate PR as it's issue of EN docs? (add), I've already done this link move in docs/zh, I know that by any result keeping docs sync is priority