AllenCell / AllenCell/cell-feature-explorer

remove firebase type from app

Offen
#139 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Tech debt
Vorherrschende Sprache
TypeScript
Sterne
2
Forks
1
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.