Typescript: make arguments optional
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- api
Línea de trabajo
Abre interface.d.ts e inspecciona las declaraciones de callback onStart, onError, onSuccess, onProgress y beforeUpload. Comprueba las definiciones de TypeScript existentes y las expectativas de uso; después, verifica que los callbacks puedan omitir parámetros no utilizados sin errores de tipos y que las declaraciones sigan siendo válidas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In interface.d.ts in these methods all the arguments are required:
onStart?: (file: RcFile) => void;
onError?: (error: Error, ret: object, file: RcFile) => void;
onSuccess?: (response: object, file: RcFile, xhr: object) => void;
onProgress?: (event: UploadProgressEvent, file: RcFile) => void;
beforeUpload?: (file: RcFile, FileList: RcFile[]) => BeforeUploadFileType | Promise<void | BeforeUploadFileType>;
But in my code I might be not interested in file argument, for example. While all args are required I must still define them and get TS errors for defined and not used variables.
Could you please make these args optional like this:
onSuccess?: (response?: object, file?: RcFile, xhr?: object) => void;
- Lenguaje dominante
- TypeScript
- Estrellas
- 804
- Forks
- 323
- Merge medio
- 12 min
- PR fusionados (30 d)
- 1
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de react-component/upload
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
react-component/upload#170 ·
-
HVAC website Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 10/100
react-component/upload#697 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 15/100
react-component/upload#615 ·
-
开启文件夹上传时就不能多选文件上传,可以解决吗? Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
react-component/upload#522 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
react-component/upload#515 ·
Todos los issues de react-component/upload
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
copse-dev/agent-pane#2953 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
[Bug]: Matrix progress drafts fail with "Matrix runtime not initialized" during tool activity Abiertobug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Client support matrix inclusion Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
calcite-components needs triage refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Esri/calcite-design-system#15203 ·