-
Notifications
You must be signed in to change notification settings - Fork 0
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
Batching Messages on Initial Project Load #71
Comments
@martin-lysk is this a side effect of the every-500-messages sleep during load? b.t.w. I could not repro this with load-test
|
@martin.lysk, @loris.sigrist please confirm that you have a repro for the issue that this is intended to fix. Thanks |
@martin-lysk does this mean that you can repro the issue by modifying messages after the first load? |
We don't have a reliable repro for the original issue, but we can repro the behavior where when making |
ok - that sounds like an API change similar to what we did for the lintreports `getAll()` api. I'll change the description of MESDK-111 accordingly. |
Context
As discussed with @martin.lysk1
Currently when a large project is loaded the
project.query.messages.getAll
gets triggered several times with intermediate message states instead of just the final message state.This causes unnecessary code executions & caching issues in dependent Inlang Apps
Proposal
Batch messages & only trigger the
getAll
callback once all messages have been loaded.The text was updated successfully, but these errors were encountered: