1. 17 Oct, 2023 2 commits
  2. 13 Oct, 2023 2 commits
  3. 12 Oct, 2023 3 commits
  4. 11 Oct, 2023 1 commit
  5. 10 Oct, 2023 5 commits
  6. 09 Oct, 2023 1 commit
  7. 06 Oct, 2023 2 commits
  8. 05 Oct, 2023 3 commits
  9. 04 Oct, 2023 1 commit
  10. 03 Oct, 2023 2 commits
  11. 02 Oct, 2023 1 commit
  12. 28 Sep, 2023 4 commits
  13. 27 Sep, 2023 3 commits
  14. 26 Sep, 2023 4 commits
  15. 14 Sep, 2023 2 commits
  16. 13 Sep, 2023 1 commit
  17. 12 Sep, 2023 1 commit
  18. 08 Sep, 2023 2 commits
    • Przemyslaw Kaminski's avatar
      058fa2fe
    • Przemyslaw Kaminski's avatar
      [ngrams] fixes to ngrams state synchronization · 09b2a0e7
      Przemyslaw Kaminski authored
      The issue was this:
      
      When user clicks around the ngrams table, only local state is
      updated (`ngramsLocalPatch`). After he clicks 'Sychronize', this is
      uploaded to server and `ngramsVersion` number is bumped up. What
      happened is that `ngramsValidPatch` inside internal state was updated
      and so the patch was sychronized locally, without refreshing the
      table. However, this resulted in errors later, when user changed the
      same term, because then, when user changes view to 'Stop terms', the
      table is refreshed with the newly applied terms and `ngramsValidPatch`
      doesn't make sense anymore.
      
      The solution is to set `ngramsValidPatch` to `mempty` and, after state
      is updated, refresh the current table.
      
      It could be argued whether it's the most optimal thing to do and why
      don't we keep things "offline". Well, clicking on 'Sychronize'
      requires the client to be online so why not sync the table already at
      this time?
      
      I guess this makes things bit simpler and I think it renders
      `ngramsValidPatch` unnecessary. So our state could be
      simplified (patches is already a complex beast).
      09b2a0e7