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

[14.0][FIX] accounting menu not translatable #2881

Draft
wants to merge 2 commits into
base: 14.0
Choose a base branch
from

Conversation

primes2h
Copy link
Contributor

@primes2h primes2h commented Jul 25, 2022

Risolve #2885 per la v.14.0

Vedi anche #2675 (v. 12.0).

@primes2h primes2h marked this pull request as draft July 25, 2022 08:36
@primes2h primes2h marked this pull request as ready for review July 25, 2022 12:45
Copy link
Contributor

@SirTakobi SirTakobi left a comment

Choose a reason for hiding this comment

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

Al momento quindi l'unica soluzione per risolvere entrambi i problemi è quella di utilizzare la #2675 insieme alla tua PR, modificando anche il parent del menu riba. (l'ho appena fatto qui)

@SimoRubi @Borruso Ho aperto #2881 per la v. 14.0 (che corrisponde alla #2675 della 12.0) che, abbinata alla presente PR dovrebbe risolvere in modo definitivo il problema.

Sorry, @SirTakobi

Originally posted by @primes2h in #2623 (comment)

Non ho accesso ai moduli Enterprise quindi non posso verificare questa PR come @SimoRubi fece per #2675.
Non essendoci codice Python non vedo come molto affidabile una revisione del codice quindi non riesco a lasciarti una review mi dispiace.

Copy link
Contributor

@sergiocorato sergiocorato left a comment

Choose a reason for hiding this comment

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

Ho testato senza EE e mi ha creato due menu separati.

@primes2h primes2h force-pushed the 14.0-fix-accounting-menu branch from f8d29c4 to 034fca5 Compare December 6, 2023 19:06
@primes2h
Copy link
Contributor Author

primes2h commented Dec 6, 2023

Ho testato senza EE e mi ha creato due menu separati.

Hai provato in locale?
Su runboat compare un solo menù.

@francesco-ooops francesco-ooops linked an issue Dec 20, 2023 that may be closed by this pull request
3 tasks
@primes2h
Copy link
Contributor Author

Ho testato senza EE e mi ha creato due menu separati.

Hai provato in locale? Su runboat compare un solo menù.

@sergiocorato ping

Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 16, 2024
@primes2h primes2h removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 18, 2024
@primes2h primes2h requested a review from sergiocorato June 18, 2024 14:54
@primes2h primes2h force-pushed the 14.0-fix-accounting-menu branch from 034fca5 to b2102f9 Compare July 25, 2024 16:43
Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

LGTM

@TheMule71 TheMule71 mentioned this pull request Nov 8, 2024
35 tasks
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 24, 2024
@sergiocorato
Copy link
Contributor

@primes2h ho caricato questa primes2h#3 per eliminare il menu doppio residuo, che a me risulta presente (l'id account.menu_finance non sparisce se non viene cancellato, e si accoda al nuovo menu_account) vedi screen di test:

Screenshot(114)

@primes2h
Copy link
Contributor Author

primes2h commented Nov 28, 2024

@primes2h ho caricato questa primes2h#3 per eliminare il menu doppio residuo, che a me risulta presente (l'id account.menu_finance non sparisce se non viene cancellato, e si accoda al nuovo menu_account)

Grazie mille @sergiocorato. Non se se avevi visto ma per la 16.0 sto implementando un nuovo approccio, perché quello attuale è decisamente problematico (vedi #4298 (comment) e #4298 (comment)) con possibili effetti collaterali non totalmente previdibili.
In particolare quando altri moduli OCA vanno ad aggiungere sottomenù ad account_menu_finance (es. account_credit_control) ti ritroveresti sempre con il doppio menù.
Con il modulo delle RiBa una problematica simile (vs. Enterprise) è stata risolta modificando il modulo stesso.
In questo caso però vorrebbe dire andare a modificare moduli OCA di altri repo per risolvere una problematica di una specifica localizzazione.

L'idea sarebbe quella di applicare il nuovo approccio anche nella 14.0, cosa ne pensi?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 1, 2024
@SirAionTech SirAionTech added the is porting This pull request is porting a change from another version label Jan 8, 2025
Copy link
Contributor

@SirAionTech SirAionTech left a comment

Choose a reason for hiding this comment

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

Da allineare a #4298.

@sergiocorato
Copy link
Contributor

@primes2h ho caricato questa primes2h#3 per eliminare il menu doppio residuo, che a me risulta presente (l'id account.menu_finance non sparisce se non viene cancellato, e si accoda al nuovo menu_account)

Grazie mille @sergiocorato. Non se se avevi visto ma per la 16.0 sto implementando un nuovo approccio, perché quello attuale è decisamente problematico (vedi #4298 (comment) e #4298 (comment)) con possibili effetti collaterali non totalmente previdibili. In particolare quando altri moduli OCA vanno ad aggiungere sottomenù ad account_menu_finance (es. account_credit_control) ti ritroveresti sempre con il doppio menù. Con il modulo delle RiBa una problematica simile (vs. Enterprise) è stata risolta modificando il modulo stesso. In questo caso però vorrebbe dire andare a modificare moduli OCA di altri repo per risolvere una problematica di una specifica localizzazione.

L'idea sarebbe quella di applicare il nuovo approccio anche nella 14.0, cosa ne pensi?

Ok, vedo che poi è stata già mergiata per la 16.0.
Riesci a farlo?

@primes2h primes2h marked this pull request as draft February 26, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
14.0 is porting This pull request is porting a change from another version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Menù "Accounting" non traducibile.
7 participants