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

Nombre del Plan de suscripción en la Factura #885

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

fproldan
Copy link
Owner

https://github.com/fproldan/DiamoERP/issues/723

Captura de pantalla 2024-10-18 a la(s) 4 16 56 p  m

for sales_invoice_item in frappe.get_all("Sales Invoice Item", {"subscription_plan": ["!=", ""]}, ["name", "subscription_plan"]):
    frappe.db.set_value("Sales Invoice Item", sales_invoice_item["name"], "subscription_plan_name", sales_invoice_item["subscription_plan"])
    frappe.db.commit()

@fproldan
Copy link
Owner Author

@ValentinaPruvost esto esta para probar en demo2 y testserviplaga.
Agregué un campo tipo Data con el nombre del plan. No entendi bien el problema original del cliente, si solo con eso ya se le soluciona el problema o hace falta algo mas.

@ValentinaPruvost
Copy link
Collaborator

@fproldan no me di cuenta de aclararlo en el issue, el problema era que el nombre del Plan se actualizaba en la factura cuando renombraban el Plan, lo que necesitan es que el nombre quede tal cual estaba cuando se creó la factura

@fproldan
Copy link
Owner Author

@ValentinaPruvost

Claro, pero eso va a seguir pasando ahora en el campo "Plan de Suscripcion" el que se deberia mantener igual siempre es el nuevo campo "Nombre del Plan de Suscripcion", no me queda claro si eso es suficiente, no se si usaban el campo en algun formato de impresion y deben cambiarlo al nuevo campo por ejemplo

@ValentinaPruvost
Copy link
Collaborator

@fproldan claro, habría que hacer que en la impresión de Factura electrónica tome el "Nombre del Plan de Suscripcion".

Igual recién probé en demo2 y sigue estando solo el enlace https://demo2.diamo.com.ar/app/sales-invoice/AM-NF-00000564

@ValentinaPruvost ValentinaPruvost merged commit ef78eb3 into diamoerp-develop Oct 21, 2024
0 of 7 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 21, 2024
* feat: subscription plan

* feat: ocultar plan en impresion
@ValentinaPruvost ValentinaPruvost deleted the issue_723 branch October 21, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants