apps-store icon indicating copy to clipboard operation
apps-store copied to clipboard

[FIX] website_apps_store: avoid useless postgres requests

Open petrus-v opened this issue 5 months ago • 4 comments

This code were calling super but overwriting most of the result.

So, stop calling super() and reuses lastest odoo version as well which stop calling search_count

This is done while managing performance issue on OCA community instance cf run 2 on locust report here: https://github.com/OCA/oca-custom/pull/62

petrus-v avatar Aug 28 '25 20:08 petrus-v

/ocabot rebase

thomaspaulb avatar Oct 03 '25 12:10 thomaspaulb

Sorry @thomaspaulb you are not allowed to rebase.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

OCA-git-bot avatar Oct 03 '25 12:10 OCA-git-bot

@thomaspaulb I've start to take care @rousseldenis suggestion and orther details during code sprint but haven't pushed yet ! but not sure to be able to do it in the next 15 days...

petrus-v avatar Oct 03 '25 13:10 petrus-v

Yes i tried the rebase for another reason, that i pinged you on the task, but it would not have helped i think. So take your time!

3 okt. 2025 15:51:48 Pierre Verkest @.***>:

 [Afbeelding]*petrus-v* left a comment (OCA/apps-store#96)[https://github.com/OCA/apps-store/pull/96#issuecomment-3365760919]

@thomaspaulb[https://github.com/thomaspaulb] I've start to take care @rousseldenis[https://github.com/rousseldenis] suggestion and orther details during code sprint but haven't pushed yet ! but not sure to be able to do it in the next 15 days...

— Reply to this email directly, view it on GitHub[https://github.com/OCA/apps-store/pull/96#issuecomment-3365760919], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AALF75CCHJT4LTTHJASOFGL3VZ5OXAVCNFSM6AAAAACFCS6KPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNRVG43DAOJRHE]. You are receiving this because you were mentioned. [Tracking afbeelding][https://github.com/notifications/beacon/AALF75HMELYNZ67B3YKPEBT3VZ5OXA5CNFSM6AAAAACFCS6KPOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTWITVXZO.gif]

thomaspaulb avatar Oct 03 '25 13:10 thomaspaulb