Evitar pago de horas a uno mismo

resuelto

#1

Cuando entras como usuario normal, si vas a ver tu perfil en la columna de ofertas, ves un icono de relojito para cada oferta publicada para poder realizar un pago por esa oferta, eso te permite pagarte horas a ti mismo, hay que quitar esos iconos (cuando se trata de tu propio perfil) y en general evitar el pago de horas a uno mismo.


#2

Podrias poner un screenshot para los que no conozcan bien el UI?

Merci!


#3

De todas formas aqui hay dos tareas:

  1. quitar el relojito de las vistas cuando es tu mismo perfil
  2. anadir en la capa de authorization (Pundit) un checl para que no pueda pasar a nivel de backend

#4

Pull request WIP https://github.com/coopdevs/timeoverflow/pull/231


#5

:clap: Merciiii :heart_eyes:


#6

Como está este tema?


#7

Un poco verde. Con el alpha de Katuma lo tuve que dejar de lado y seguiré con ello hasta que saquemos el alpha almenos.

Intentaré dejarlo arreglado y explicado para que alguien más lo pueda retomar


#8

Al final dejarlo arreglado y acabarlo es lo mismo, así que he optado por lo segundo.

revisar los cambios https://github.com/coopdevs/timeoverflow/pull/231. Solo me queda persistir el Transfer.


#9

Duda @sseerrggii. Ahora mostramos el mensaje de error que adjuntado al PR, pero deberíamos deshabilitar de alguna manera el formulario, antes de que el usuario lo rellene por UX? tipo poner los campos de color gris. Como lo hemos hecho hasta ahora en otras partes?


#10

Yo creo que no hace falta @saulopefa , con mostrar un error seria suficiente

En cuanto este merge lo testeo y comento si hay alguna cosa.

Gracias!


#11

Entonces, cual es el procedimiento? merge a develop + test manual?


#12

Si, de momento yo testeo develop en local y cuando está ok ya va para producción


#13

Bueno, bueno… una PR la tenemos que testear antes de que entre en develop :pray:

@sseerrggii por favor haz git fetch y git checkout RAMA_QUE_HAY_QUE_QA y pruebalo en local antes


#14

Testeado en local, todo funciona bien :+1:


#15

Pull request merged. Este cambio entrará en la próxima release.

:clap::clap::clap: Gracias a todos!


cerrado #16