search

bcardiff / themis

Other
1 stars 1 forks source link

Poder quitar del listado planes de pago obsoletos #67

Closed bcardiff closed 1 year ago

bcardiff commented 2 years ago

El listado de precios está un poco cargado.

Supongo que sería necesario poder marcar como obsoletos algunos planes de pago. O al menos no mostrar aquellos que tiene precio $0.

Screen Shot 2022-03-26 at 18 09 44

Qué planes se muestran se determina en el siguiente lugar:

https://github.com/bcardiff/themis/blob/11bebaf69e928fa4895257bb961a9912d291c215/app/helpers/application_helper.rb#L109-L111

@manuelbicain, opiniones?

manuelbicain commented 2 years ago

Sep. Los obsoletos los dejé en 0 pensando en eso.

El sáb, 26 mar 2022 a las 18:13, Brian J. Cardiff @.***>) escribió:

El listado de precios está un poco cargado.

Supongo que sería necesario poder marcar como obsoletos algunos planes de pago. O al menos no mostrar aquellos que tiene precio $0.

[image: Screen Shot 2022-03-26 at 18 09 44] https://user-images.githubusercontent.com/459923/160257233-4b177cd1-ed60-4ee4-946c-700e28f149e4.png

Qué planes se muestran se determina en el siguiente lugar:

https://github.com/bcardiff/themis/blob/11bebaf69e928fa4895257bb961a9912d291c215/app/helpers/application_helper.rb#L109-L111

@manuelbicain https://github.com/manuelbicain, opiniones?

— Reply to this email directly, view it on GitHub https://github.com/bcardiff/themis/issues/67, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC72TLDBGWPVUCTTG4O262LVB54XJANCNFSM5RXVSBWQ . You are receiving this because you were mentioned.Message ID: @.***>

Mar12358 commented 1 year ago

Quitar de seeds.rb payment_plan obsoletos. Además de eso, entiendo que quieren que los pueda quitar el usuario admin directamente desde la app, en ese caso se podría implementar dsps. simplemente creando una función q acceda a seeds y elimine la línea en cuestión. Pero por el momento quitar de seeds. @bcardiff está bien lo que estoy pensando? O es que quieren conservarlos y marcarlos como obsoletos para después filtrarlos out...?

manuelbicain commented 1 year ago

Ojo, entiendo que está corrido el último seeds. Mirá la fecha, lo subió Javi hace dos meses aprox.

El mar, 14 feb 2023 a las 10:44, Mar12358 @.***>) escribió:

Quitar de seeds.rb payment_plan obsoletos. Además de eso, entiendo que quieren que los pueda quitar el usuario admin directamente desde la app, en ese caso se podría implementar dsps. simplemente creando una función q acceda a seeds y elimine la línea en cuestión. Pero por el momento quitar de seeds. @bcardiff https://github.com/bcardiff está bien lo que estoy pensando?

— Reply to this email directly, view it on GitHub https://github.com/bcardiff/themis/issues/67#issuecomment-1429770133, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC72TLHWXGQJ2D6MQNQAE43WXOD4BANCNFSM5RXVSBWQ . You are receiving this because you were mentioned.Message ID: @.***>

Mar12358 commented 1 year ago

Sí, ahí lo veo.. pero modificó algunos profes, algunas clases de belgrano, algun que otro precio de algun pack, Faltaría quitar de ahí mismo los packs obsoletos. Lo dejo en el hilo asentado así dsps cuando empiece a resolver los issues lo tengo cocinado. Por ahora sigo viendo el código...

manuelbicain commented 1 year ago

Creo que el problema es que si sacamos los packs obsoletos del seeds no te lo elimina del GLIMS.

El mar, 14 feb 2023 a las 11:00, Mar12358 @.***>) escribió:

Sí, ahí lo veo.. pero modificó algunos profes, algunas clases de belgrano, algun que otro precio de algun pack, Faltaría quitar de ahí mismo los packs obsoletos. Lo dejo en el hilo asentado así dsps cuando empiece a resolver los issues lo tengo cocinado. Por ahora sigo viendo el código...

— Reply to this email directly, view it on GitHub https://github.com/bcardiff/themis/issues/67#issuecomment-1429791927, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC72TLD2GUHQ7WSGI5WNG4LWXOFYBANCNFSM5RXVSBWQ . You are receiving this because you were mentioned.Message ID: @.***>

Mar12358 commented 1 year ago

bueno, ahí comenté los packs de afro como prueba, para ver eso... y claro, solo encuentra o crea, si saco algo no hace nada, hay que explicitamente destruirlo usando el método destroy de ActiveRecord en el modelo PaymentPlan, O simplemente agregando un atrivuto que podría llamarse "visible" y dsps los filtramos desde acá: https://github.com/bcardiff/themis/blob/11bebaf69e928fa4895257bb961a9912d291c215/app/helpers/application_helper.rb#L109-L111

Mar12358 commented 1 year ago

Listo, Aparecen los packs que no tienen precio = 0, excepto aquellos cuya descripción incluya "bonif", admite upper/lower case por igual. Cierro Issue