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

[ADD] New module: <sale_order_line_propagate_date> #104

Closed
wants to merge 2 commits into from

Conversation

esthermm
Copy link
Contributor

Modificar sale_order_line view para que visualice el campo requested_date.
Añadir un shortcut desde clientes a las lineas de pedido de venta
Al generar el albarán desde el pedido, las líneas de albarán en fecha planificada deberían llevar el requested date de la línea de pedido y el albarán en sí, llevaría la menor de las fechas de todas sus líneas.

@oihane
Copy link
Contributor

oihane commented Oct 30, 2015

Haz un rebase

@esthermm
Copy link
Contributor Author

esthermm commented Nov 2, 2015

@pedrobaeza me da fallo el runbot porque al parecer falta un módulo, pero @oihane me ha comentado que debería de estar corregido. ¿Se te ocurre alguna solución?

@pedrobaeza
Copy link
Member

Faltaba esa dependencia. Ahora debe salir bien

@esthermm
Copy link
Contributor Author

esthermm commented Nov 3, 2015

@pedrobaeza todo OK. 👍

==============================


This module add the requested_date to sale order lines tree view and creates a
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/add/adds

@pedrobaeza
Copy link
Member

Aquí veo dos funcionalidades distintas que no tienen nada que ver: por un lado, lo de la fecha, que es el principal motivo del módulo y bien. Pero por otro, lo de las líneas de pedido de venta, es algo que ya está en otro módulo que añade un menú de líneas de pedido de venta, y además, no le veo mucho sentido teniendo el one2many en el propio formulario. Esa funcionalidad debería quitarse.

@esthermm
Copy link
Contributor Author

esthermm commented Nov 3, 2015

Ya he quitado esa funcionalidad para llevarla al otro módulo y esta corregida la versión

@anajuaristi
Copy link
Contributor

Veo correcto que este módulo lleve solo el tema fechas y que el shorcut a líneas de pedido desde el cliente, se meta en el mismo módulo que crea el menú de líneas de pedido.

Probada en runbot la siguiente funcionalidad:
.- Cuando cambias requested date en el pedido --> modifica en todas las líneas del pedido OK
.- Cambiando las líneas de forma independiente, al crear el albarán arrastra cada una: OK
.- El albarán lleva la menor fecha de todas las líneas: OK.

Por tanto, por mi parte 👍 RTM

@oihane
Copy link
Contributor

oihane commented Nov 3, 2015

Cuando cambias el requested date en el pedido que modifique las líneas no lo hace este modulo

@pedrobaeza
Copy link
Member

A raiz del comentario de Oihane, por qué se ha hecho otro módulo distinto para esto teniendo https://github.com/odoomrp/odoomrp-wip/blob/8.0/sale_order_line_dates/models/sale_order.py?

@esthermm
Copy link
Contributor Author

esthermm commented Nov 3, 2015

No, no lo hace este módulo, lo hace el sale_order_line_dates, por eso le he añadido la dependencia pero si cambias a mano las fechas de la líneas del pedido de venta, no se las lleva al albarán, por eso se ha realizado este desarrollo.

@pedrobaeza
Copy link
Member

Pero eso, mejor incluir eso en el mismo módulo

@esthermm esthermm closed this Nov 4, 2015
@esthermm esthermm deleted the 8.0-sales branch November 4, 2015 09:24
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.

4 participants