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

[16.0][FIX] l10n_es_partner: Fix vista formulario banco #3735

Conversation

mariancuadranetkia
Copy link

Arregla vista formulario de banco.
Cómo se ve sin el cambio:
antes_nuevo

Cómo se ve después del cambio:
despues_nuevo

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@sergio-teruel
Copy link
Contributor

Si pones los campos así, recupera todo el ancho...
image

@mariancuadranetkia
Copy link
Author

@sergio-teruel Sería una buena solución cambiar el número de columnas del grupo bank_details? De esa manera se vería bien el campo nombre completo...
imagen

Gracias por la ayuda!!

Copy link
Contributor

@sergio-teruel sergio-teruel left a comment

Choose a reason for hiding this comment

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

¿Puedes eliminar los atributos select? Creo que ya no se usan....

@sergio-teruel
Copy link
Contributor

Gracias Marian.

@pedrobaeza
Copy link
Member

@mariancuadranetkia haz por favor squash de los 3 commits.

[FIX] l10n_es_partner: Fix vista formulario banco recuperando el ancho

[FIX] l10n_es_partner: Fix vista formulario banco, respetando ancho y eliminando select attribute.
@mariancuadranetkia mariancuadranetkia force-pushed the 16.0-fix-l10n_es_partner-bank_view_form branch from 48556de to 134b14f Compare October 1, 2024 05:28
@mariancuadranetkia
Copy link
Author

@pedrobaeza Hecho!
Gracias a todos, por vuestras sugerencias y ayuda

@pedrobaeza pedrobaeza added this to the 16.0 milestone Oct 1, 2024
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-3735-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit fbc07fb into OCA:16.0 Oct 1, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 47912fe. Thanks a lot for contributing to OCA. ❤️

@mariancuadranetkia mariancuadranetkia deleted the 16.0-fix-l10n_es_partner-bank_view_form branch October 1, 2024 05:45
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.

5 participants