sahat/satellizer

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

Open

#977 aperta il 24 set 2016

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)TypeScript (1110 fork)github user discovery
hacktoberfest

Metriche repository

Star
 (7803 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca nel codice tutte le occorrenze di `this.$window['cordova']` e aggiorna ogni condizione per gestire anche il caso in cui `cordova.platformId === 'browser'` in ambienti Ionic. Ciò consentirà alla libreria di funzionare correttamente in un browser quando viene servita da un server dopo una build Ionic.
Tech stack
javascriptangular
Dominio
authenticationfrontendmobile dev
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
AngularJSGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75