You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.
Hi, thanks for this project. I am realized shortid is listed deprecated in their package.
shortid is deprecated, because the architecture is unsafe.
They do recommend using Nano ID and the bundle size is smaller according to Bundlephobia.
I am not sure will it have any impact if change from shortid to any other similar package.
The text was updated successfully, but these errors were encountered:
@tehkapa The shortid uses a dynamic length id. But the nanoid use a fixed-length id. So, after migrate to nanoid, the question is, what length should we use? I think 9 or 10 is enough. Here is a collision calculator: Nano ID Collision Calculator
@tehkapa
Finally, I found that the nanoid above 3.0 met a bug in Create React App, which only fixed after Create React App 4.0.
See: nanoid doc about Create React App.
We have two choices:
Use the nanoid 2.0 first, and upgrade it to the latest version after we upgrade the react-scripts.
Keep using shortid, migrate to nanoid after we upgrade the react-scripts.
Hi, thanks for this project. I am realized shortid is listed deprecated in their package.
They do recommend using Nano ID and the bundle size is smaller according to Bundlephobia.
I am not sure will it have any impact if change from shortid to any other similar package.
The text was updated successfully, but these errors were encountered: