frlp-utn-ingsoft / orderingg

Aplicación utilizada en la cursada 2018 de Ingenieria de Software. UTN-FRLP
4 stars 20 forks source link

Problema en la aplicacion orderingg #43

Open ignacio96 opened 6 years ago

ignacio96 commented 6 years ago

Corro la aplicacion pero no me aparece la tabla, tampoco se abre el modal al hacer click en agregar. Cuando hice el merge para actualizar la rama master y me saltaron los conflictos me quede con lo de la rama upstream/master. Puede ser que sea por eso?

RodrigoJacznik commented 6 years ago

Si te quedaste con nuestros cambios debería funcionar. Tenes algún error en la consola del browser?

ignacio96 commented 6 years ago

Si me tira dos errores, uno en api.js y otro en index.js

RodrigoJacznik commented 6 years ago

subí acá los errores así te ayudo a solucionarlos

ignacio96 commented 6 years ago

y como hago eso? te paso el mensaje de error que tira y la linea en la que esta?

RodrigoJacznik commented 6 years ago

Si

ignacio96 commented 6 years ago

Uncaught SyntaxError: Unexpected token function api.js:18

Uncaught ReferenceError: API is not defined index.js:6 at index.js:6 at index.js:146

RodrigoJacznik commented 6 years ago

Pasame desde la línea 10 a la 25 del archivo api.js

ignacio96 commented 6 years ago
            return r.json();
        });
}

/**
 * Obtiene todos los productos desde el backend
 *
 */
function getProducts() {
    return fetch('/product')
        .then(function toJson(r) {
            return r.json();
        });
}

/**
RodrigoJacznik commented 6 years ago

No veo el error. Podes hacer un branch, pushearlo y pasarme el link así puedo verlo completo?

ignacio96 commented 6 years ago

https://github.com/luchoCap/orderingg/tree/problema-aplicacion-orderingg

RodrigoJacznik commented 6 years ago

Fijate que en la linea 1 de api.js le falta un paréntesis. Debería ser: (function getOrder(orderId) {

Avísame si se soluciono el problema, en un rato te puedo hacer un PR de ultima.

ignacio96 commented 6 years ago

No se soluciono, sigue tirando los mismos errores

ignacio96 commented 6 years ago

@RodrigoJacznik Alguna novedad?

RodrigoJacznik commented 6 years ago

Fíjate que el contenido de api.js sea el mismo al de este archivo: https://raw.githubusercontent.com/frlp-utn-is-2018/orderingg/dev/app/static/js/api.js

Recién baje su repo y probé ese cambio y funciona bien. Se hicieron lio al mergear.