Code review comment for lp:~wgrant/launchpad/tm-suggest-constant

Revision history for this message
Celso Providelo (cprov) wrote :

Nice change, introducing preloadPOFilesAndSequence really solves the problem for loading suggestions for a given message.

We can land it ASAP.

(thinking out loud here, feel free to ignore me)

Although, I am not seeing a clear path for going further and pre-loading this for multiple messages. Maybe we won't need it (this step will bring enough benefits) or we could try to investigate something at the model level for making it possible.

review: Approve

« Back to merge proposal