sahat/satellizer

The library thinks that when we are on a browser cordova will be undefined.

Open

#977 geöffnet am 24. Sept. 2016

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (1.110 Forks)github user discovery
hacktoberfest

Repository-Metriken

Stars
 (7.803 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I think the reason that the library does not work on a browser with IONIC / cordova is the fact that it assumes that when we are on a browser this.$window['cordova'] will be undefined.

But actually this is not the case in an ionic project. On a browser platform, after we build and upload the application on a server cordova exists! So, if I am right and please correct me if I am wrong, in order to be able to use the library, you will need to replace in every if else condition (this.$window['cordova'] && this.$window['cordova'].platformId === "browser) in order to be able to work on a browser platform.

Contributor Guide

Research-Richtung
Durchsuchen Sie die Codebasis nach allen Vorkommen von `this.$window['cordova']` und aktualisieren Sie jede Bedingung, um auch den Fall zu behandeln, wenn `cordova.platformId === 'browser'` in Ionic Umgebungen. Dadurch kann die Bibliothek in einem Browser korrekt funktionieren, wenn sie nach einem Ionic Build von einem Server bereitgestellt wird.
Tech Stack
javascriptangular
Domain
authenticationfrontendmobile dev
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
AngularJSGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75