-
Notifications
You must be signed in to change notification settings - Fork 0
Major update refactoring library using Vite #10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…playground also with Vite
…tore into feat/vite-esm
| { | ||
| "name": "@ensuro/api-calls-store", | ||
| "version": "%%VERSION%%", | ||
| "description": "API Calls Store for React Redux", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Queda sin descripción?
react-playground/README.md
Outdated
| @@ -0,0 +1,12 @@ | |||
| # React + Vite | |||
|
|
|||
| This template provides a minimal setup to get React working in Vite with HMR and some ESLint rules. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cambiar este README, a algo relacionado con el proyecto.
react-playground/index.html
Outdated
| <meta charset="UTF-8" /> | ||
| <link rel="icon" type="image/svg+xml" href="/vite.svg" /> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
| <title>Vite + React</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
El título de la página también, api-calls-store playground o algo así.
react-playground/src/store/index.js
Outdated
| getDefaultMiddleware({ | ||
| thunk: false, | ||
| serializableCheck: false, | ||
| immutableCheck: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Qué son estos check? Capaz que está bueno empezar a usarlos.
src/package-index.js
Outdated
| } | ||
| getAPIFn = getAPI; | ||
| defaultCount = clockCount || 10; | ||
| if (typeof clockCount === "number") defaultCount = clockCount; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if clockCount !== undefined sería más claro.
No description provided.