AllenCell / AllenCell/cell-feature-explorer

remove firebase type from app

Abierto
#139 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Tech debt
Lenguaje dominante
TypeScript
Estrellas
2
Forks
1
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

## Use Case

All firebase types should be hidden behind the image dataset interface but one has leaked out: the `dateCreated` in state/image-dataset/types.ts is a `Timestamp` which is a firebase type.

## Solution

Change the runtime definition of dateCreated to be a regular Date or number... and update the firebase dataset implementation to convert the value.

## Alternatives

Have firebase return a number instead of a Timestamp?

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.