miguel-perez/smoothState.js

Ajax success method issue

Open

#229 aperta il 5 nov 2015

Vedi su GitHub
 (4 commenti) (1 reazione) (0 assegnatari)CSS (515 fork)batch import
bughelp wanted

Metriche repository

Star
 (4433 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hello,

My implementation has been choking on line 375 letting me know that $.ajax(settings).success(...) is not a function. After researching it seems that the success / error / complete callbacks can be defined in the settings object but there is no such function, the closest being done().

So I changed the source code to settings.success = function(...) and the same for error and complete, before calling $.ajax(settings).

It seems to be working. But not being a developper I'm not really sure that would be the appropriate fix...

Guida contributor

Direzione di ricerca
Il problema è che $.ajax().success() è deprecato in jQuery 1.8+. Sostituiscilo con $.ajax().done() o definisci success nell'oggetto settings. Individua la riga in cui viene chiamato $.ajax(settings).success() e modificala per usare done() o sposta il callback nell'oggetto settings.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80