Skip to content
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

UserController para manejar roles #45

Open
wants to merge 13 commits into
base: dev
Choose a base branch
from

Conversation

mapacheverdugo
Copy link
Member

@mapacheverdugo mapacheverdugo commented Oct 16, 2023

Descripción

La idea es armar el camino para poder manejar un menú dinámico para los usuarios, por ejemplo existirán usuarios con beca de alimentación y otros sin, eso deberá quedar reflejado en los roles. Otro caso a manejar es a los usuarios con múltiples carreras.

  • Se movió todo lo relacionado al usuario a un nuevo controller de usuario
  • Se movió el selectedCarrera a este controller de usuario
  • Se agregó el modelo Role con algunos roles predefinidos
  • Se agregó modelo para el drawer item, y se actualizó lo relacionado en Remote Config

Lo que hay que probar es que todo funcione como siempre ha funcionado

Tipo de cambio

  • Corrección de error.
  • Nueva funcionalidad .
  • Breaking change (cambio que puede romper la aplicación).
  • Documentación.
  • Mejora de codigo existente (performance y/o estilo del codigo y/o otra mejora)

Checklist:

  • He hecho una revisión de mi propio codigo.
  • Hice pruebas en un dispositivo Android real.
  • Hice pruebas en un dispositivo Android emulador.
  • Hice pruebas en un dispositivo iOS real.
  • Hice pruebas en un dispositivo iOS simulador.
  • Mis cambios no generan nuevos lint Problems.
  • Agregué mis cambios al CHANGELOG.md.
  • Este pull request contiene <1000 lineas de codigo (LOC).

Recursos

Añade cualquier documentación que facilite la comprensión de la solución.

@RafaelMoralesV
Copy link
Member

Duda, la parte de información de los roles la trae la api de sisei me imagino, no?

@mapacheverdugo
Copy link
Member Author

No xd, son roles propios que se "calculan" según la info que tengamos

Quizás "role" no es el nombre más adecuado, pero por ej. yo tengo que obtener todas las carreras del usuario y luego verifico si entre sus carreras hay una en estado "Regular" para asignarle el rol hasActiveCarrera. Lo mismo pasará con la beca de alimentación, yo intentaré obtener los cupones de la beca de alimentación de la persona, y si nuestra API responde que la persona no tiene la beca, no se le agregará el rol de la beca.

@RafaelMoralesV
Copy link
Member

dale, entiendo

Quizás sea mejor usar un término como 'estado'. Role me da a entender algo como si el usuario es profesor, o si es estudiante, no sé ñdkdjdbd

* Se han movido algunos archivos a sus carpetas correspondientes para tener un mejor orden.
* Se ha reparado un error (por borrar la inicialización del controlador de la calculadora)

Signed-off-by: Tomás L <[email protected]>
@Im-Fran
Copy link
Member

Im-Fran commented Oct 17, 2023

Lol, ni idea por que sale como firmado por Tomás pero weno, hice lo que dice el commit, arreglé unas cositas y ordené unas clases de la app

@RafaelMoralesV
Copy link
Member

Lol, ni idea por que sale como firmado por Tomás pero weno, hice lo que dice el commit, arreglé unas cositas y ordené unas clases de la app

Si lo hiciste desde el mac del club, es posible que la wea tenga vinculada una llave GPG del Tomás (?

@Im-Fran
Copy link
Member

Im-Fran commented Oct 18, 2023

En realidad creo que es porque habilité el Sign off, pero es lo de menos :p

@mapacheverdugo
Copy link
Member Author

@Im-Fran no sigas commiteando a mi rama que pierdo el control de mi PR, y si ya todo te funciona dale a la aprobación pls

@Im-Fran
Copy link
Member

Im-Fran commented Oct 20, 2023

Ok, creo que fuera del error que arreglé está todo bien

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants