Community build #002 #3

Merged
gozzy merged 0 commits from staging into master 2022-09-22 10:18:59 +03:00
  • Reduced event cache size by depreciating the unecessary "zip-local" package over the already existing "jszip" package, which wasn't being used for compression. It also allowed to maximising the compression ratio which resulted in a 1% compression gain over the legacy method.

  • Improved the deposit modal UI disclaimer about note backups in place of suggesting on-chain note recovery - which in of itself is a privacy contingency.

  • Resolved event caching detection issues, as the array.indexOf query was giving false positives since the bool condition only enforced !== 0. When in actually an incorrect query returns -1, decision was to replace it with array.includes.

  • Resolved RPC fetching conflicts, as the lastBlock reference from getEventsFromDB is incorrectly indexed as IndexedDB does not maintain assortment. This was resolved by forcing sorting events as they are originally sorted each time it is queried, which isn't ideal but shall be improved in future updates.

* Reduced event cache size by depreciating the unecessary "zip-local" package over the already existing "jszip" package, which wasn't being used for compression. It also allowed to maximising the compression ratio which resulted in a 1% compression gain over the legacy method. * Improved the deposit modal UI disclaimer about note backups in place of suggesting on-chain note recovery - which in of itself is a privacy contingency. ![](https://i.imgur.com/hOS1yxq.png) * Resolved event caching detection issues, as the `array.indexOf` query was giving false positives since the bool condition only enforced `!== 0`. When in actually an incorrect query returns `-1`, decision was to replace it with `array.includes`. * Resolved RPC fetching conflicts, as the `lastBlock` reference from `getEventsFromDB` is incorrectly indexed as IndexedDB does not maintain assortment. This was resolved by forcing sorting events as they are originally sorted each time it is queried, which isn't ideal but shall be improved in future updates.
Sign in to join this conversation.
No Reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tornadocash/classic-ui#3
No description provided.