search

Sinapse-Energia / APP-Installation

Android APP for trace devices
0 stars 0 forks source link

[FTP] To change hardcoded IP to DNS #2

Open ralcaide opened 6 years ago

ralcaide commented 6 years ago

Now, the connection to the FTP server is Hardcoded to 89.248.100.11 in ./trunk/Install_sinapse/src/install/sinapse/Install_sinapseActivity.java, line 680. It is necessary to change it to DNS with the following URL:

ftp://ftp.sinapseenergia.com/

Same user and password used until now

noloman commented 6 years ago

Buenas! Os mando una primera version del APK, vale? Por favor testeadlo bien porque yo apenas se de la app y el codigo es UN INFIERNO :) Os lo dejo en un archivo ZIP porque GitHub no deja subir APKs, asi que simplemente lo descomprimis y deberiar funcionar. Un saludo y ya me direis!

app-debug.apk.zip

ralcaide commented 6 years ago

Thanks!! we will test it asap and provide you feedback :)

noloman commented 6 years ago

Buenas!

Os subo una version nueva del APK. No instalaba al parecer porque con las nuvas versiones de Android Studio, al usar Gradle, no se puede generar un APK como lo he hecho siempre, sino de una nueva forma. Lo he probado y funciona perfecto en mi telefono.

Un saludo y ya me direis!

app-debug.apk.zip

ralcaide commented 6 years ago

Tras un test preliminar se observa:

  1. Falta primera pantalla que sirve para introducir el nombre de la instalación. Este nombre es un campo del JSON y además se usa para crear la carpeta en el servidor FTP. Se podría crear una pantalla simple para introducir este nombre y que la aplicación haga el uso que necesita?
  2. "Añadir punto" no funciona correctamente. Se escanea el punto pero luego no se puede continuar
  3. "Añadir otro punto" parece que sí funciona
  4. Tras añadir varios puntos usando la opción anterior se genera parte de trabajo corréctamente pero en el servidor FTP no hay ningún fichero nuevo. Puedes conectarte y con las credenciales que usa la aplicación usando el filezilla y ver que no hay nada nuevo.

Lo ideal sería solucionar al menos puntos 1 y 4 para poder tener algo funcional

noloman commented 6 years ago

buenas!

  1. Hare esa pantalla primero; el UI no es complicado, pero tengo que ver exactamente ese tema del JSON.
  2. Deberia funcionar el codigo QR que hay subido en github de prueba? Si no, podriais darme un QR que funcione para este punto? (no se si tiene que ver con que quizas el QR no se el apropiado para que funcione el punto 3 pero no el 2).
  3. Se me ocurre que quizas no funcione, porque el punto 1 no esta completado.

En resumen: me pondre a hacer una nueva pantalla del punto 1 cuanto antes y cualquier problema os comento. Un saludo

noloman commented 6 years ago

Buenas de nuevo! Acabo de crear el UI de la primera pantalla, os dejo una captura:

screenshot 2018-10-15 at 17 03 29

Sin embargo, voy a necesitar las especificaciones funcionales para hacerlo funcionar:

ralcaide commented 6 years ago

Muchas gracias por el feedback y la pantalla. Tras darle varias vueltas hemos visto que faltan más cosas de lo que esperábamos. Deja esto parado de momento, vamos a intentar encontrar el código.

noloman commented 6 years ago

Buenas,

Claro como queráis. Avisadme cuando tengáis algo y sigo. Un saludo!

Op di 16 okt. 2018 10:01 schreef ralcaide notifications@github.com:

Muchas gracias por el feedback y la pantalla. Tras darle varias vueltas hemos visto que faltan más cosas de lo que esperábamos. Deja esto parado de momento, vamos a intentar encontrar el código.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/Sinapse-Energia/APP-Installation/issues/2#issuecomment-430140278, or mute the thread https://github.com/notifications/unsubscribe-auth/AALL4MJY2nnGoPvizllbu9c9d9uZf_NYks5ulZJjgaJpZM4XBgbp .