jsonforms icon indicating copy to clipboard operation
jsonforms copied to clipboard

fix(translations): memoize array translation

Open LukasBoll opened this issue 1 year ago • 6 comments

This commit addresses an issue where array translations were created as new objects within the core module for each render cycle, causing unnecessary rerenders. By memoizing the translation object in the material renderer set, this commit optimizes performance and prevents redundant rerenders.

LukasBoll avatar Jul 23 '24 10:07 LukasBoll

Deploy Preview for jsonforms-examples ready!

Name Link
Latest commit 0e697e75f59da1dc2a6e1e169e3e7ecd8f795698
Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/66be575ee1c5a000082d87f5
Deploy Preview https://deploy-preview-2358--jsonforms-examples.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Jul 23 '24 10:07 netlify[bot]

@LukasBoll Can you fix the build or do you need support?

sdirix avatar Aug 14 '24 09:08 sdirix

Coverage Status

coverage: 81.733% (-1.6%) from 83.285% when pulling c64196b17c42ccee60a0a97bf958dc5377337060 on LukasBoll:resolve_translation_rerender into 52b843a9743967136221f210a413139fee54b21d on eclipsesource:master.

coveralls avatar Aug 15 '24 19:08 coveralls

Deploy Preview for jsonforms-examples ready!

Name Link
Latest commit c64196b17c42ccee60a0a97bf958dc5377337060
Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/66e4636c30eae20008c99350
Deploy Preview https://deploy-preview-2358--jsonforms-examples.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Aug 15 '24 20:08 netlify[bot]

Hi @sdirix, I just fixed the build issues :)

LukasBoll avatar Aug 16 '24 06:08 LukasBoll

@LukasBoll Can you resolve the conflicts?

sdirix avatar Aug 21 '24 15:08 sdirix

Merge commit only fixes a tiny merge conflict.

lucas-koehler avatar Sep 13 '24 16:09 lucas-koehler