AllenCell / AllenCell/cell-feature-explorer

remove firebase type from app

Ouverte
#139 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Tech debt
Langage dominant
TypeScript
Étoiles
2
Forks
1
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

## 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?

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.