search

BuenosCodes / etitango-front

Front-end
4 stars 2 forks source link

feat: allow time selection on date fields #166

Open amartinezsaucedo opened 1 year ago

amartinezsaucedo commented 1 year ago

closes #165

sna1988 commented 10 months ago

@amartinezsaucedo no veo ningun motivo para no mergear este PR, avisame si queres lo mergeo yo no problem.

CC: @thehighestprimenumber

nicoduquelsky commented 9 months ago

@amartinezsaucedo no veo ningun motivo para no mergear este PR, avisame si queres lo mergeo yo no problem.

CC: @thehighestprimenumber

Entiendo que esto quedó sin mergear porque @rusonariga indicó que no era necesario en el issue #165.

Hay un caso en el que sí es necesario que es la hora de apertura de inscripciones en la creación de evento. Actualmente lo setea @thehighestprimenumber manualmente o quedan las 00hs por defecto al crear el evento.

Pregunta de total ignorancia: con este merge, ¿se podría seguir usando el componente sin selección de horario (el que existe actualmente) y también con selección de horario, según lo necesitemos en cada campo? ¿O queda un componente de fecha con selección de horario ineludiblemente?

amartinezsaucedo commented 8 months ago

@amartinezsaucedo no veo ningun motivo para no mergear este PR, avisame si queres lo mergeo yo no problem. CC: @thehighestprimenumber

Entiendo que esto quedó sin mergear porque @rusonariga indicó que no era necesario en el issue #165.

Hay un caso en el que sí es necesario que es la hora de apertura de inscripciones en la creación de evento. Actualmente lo setea @thehighestprimenumber manualmente o quedan las 00hs por defecto al crear el evento.

Pregunta de total ignorancia: con este merge, ¿se podría seguir usando el componente sin selección de horario (el que existe actualmente) y también con selección de horario, según lo necesitemos en cada campo? ¿O queda un componente de fecha con selección de horario ineludiblemente?

Se puede usar el componente con y sin selección de horario