-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Nuevo sistema Mboehao #25
Conversation
…do a problema comentado en documentacion.
…os metodos con librerias actualizadas.
…s warnings y errores arreglados.
import io.github.jokoframework.otp.OtpActivity; | ||
import io.github.jokoframework.service.TestServiceNotification; | ||
|
||
public class Home2Activity extends AppCompatActivity implements NavigationView.OnNavigationItemSelectedListener { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Debería quedar un sólo HomeActivity, el más nuevo que veo que es esta Clase. Aplicar ese cambio a todo el proyecto.
No tiene sentido mantener el que ya no funciona con las nuevas APIs, en todo caso; dejar con el mismo nombre y eliminar esta clase; para que no sea necesario cambiar en todas las clases que hacen referencia al nombre HomeActivity
.
… nueva (Home2Activity) a HomeActivity.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closes #26
Nuevas funcionalidades, implementado Material Design.
Corregido varios errores y warnings.