Closed wgxh-cli closed 5 months ago
Name | Link |
---|---|
Latest commit | 79d2abad82884c8c23fc3fb68333cd7d73763a43 |
Latest deploy log | https://app.netlify.com/sites/newcar-copy/deploys/66406bafb4d87100086af6dd |
Name | Link |
---|---|
Latest commit | 79d2abad82884c8c23fc3fb68333cd7d73763a43 |
Latest deploy log | https://app.netlify.com/sites/newcar-playground/deploys/66406bafe989420008c6ee7e |
Deploy Preview | https://deploy-preview-41--newcar-playground.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Name | Link |
---|---|
Latest commit | 79d2abad82884c8c23fc3fb68333cd7d73763a43 |
Latest deploy log | https://app.netlify.com/sites/newcar-docs/deploys/66406bafac47660008a30475 |
Deploy Preview | https://deploy-preview-41--newcar-docs.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Name | Link |
---|---|
Latest commit | 79d2abad82884c8c23fc3fb68333cd7d73763a43 |
Latest deploy log | https://app.netlify.com/sites/newcar-apis/deploys/66406baf8358c70008c76ecf |
Deploy Preview | https://deploy-preview-41--newcar-apis.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Add some comments to explain code writers' intentions, and removed some ternary operations for readability.
WARN
Too many ternary operations will break the readability, for reader's feeling, please put them in right place.
For example, when assigning for a variable when the value is optional or conditional, then ternary operations would adapt a lot. But, for example, in
patch.ts
, When doing some operations that causes side effect, you should useif else
block instead.That's my advice, take it easy!~