openmrs / openmrs-esm-form-engine-lib

React Form Engine library for O3
Other
10 stars 59 forks source link

(Fix) Use size 100 for the references chunk size to append to the url. #335

Closed hadijahkyampeire closed 3 months ago

hadijahkyampeire commented 3 months ago

Requirements

Summary

So in this case, we don't need the system default chunk size (1000) because that would me 1000 concept uuids added to the endpoint as query params. In this PR, I used a random value 100 which is efficient in performance of the request time and a good chunk to fetch in cases where we have a lot of data.

Screenshots

Screenshot 2024-07-01 at 15 05 04

Related Issue

Other

github-actions[bot] commented 3 months ago

Size Change: -245 kB (-19.51%) 🎉

Total Size: 1.01 MB

Filename Size Change
dist/800.js 0 B -245 kB (removed) 🏆
ℹ️ View Unchanged | Filename | Size | Change | | :--- | :---: | :---: | | `dist/184.js` | 11.2 kB | 0 B | | `dist/195.js` | 76.6 kB | 0 B | | `dist/225.js` | 2.57 kB | 0 B | | `dist/29.js` | 163 kB | 0 B | | `dist/300.js` | 709 B | 0 B | | `dist/327.js` | 1.84 kB | 0 B | | `dist/335.js` | 967 B | 0 B | | `dist/353.js` | 3.02 kB | 0 B | | `dist/41.js` | 3.36 kB | 0 B | | `dist/422.js` | 6.79 kB | 0 B | | `dist/474.js` | 116 kB | -2 B (0%) | | `dist/491.js` | 9.18 kB | 0 B | | `dist/540.js` | 2.63 kB | 0 B | | `dist/55.js` | 756 B | 0 B | | `dist/635.js` | 14.3 kB | 0 B | | `dist/70.js` | 482 B | 0 B | | `dist/708.js` | 245 kB | 0 B | | `dist/776.js` | 3.2 kB | 0 B | | `dist/99.js` | 690 B | 0 B | | `dist/993.js` | 3.09 kB | 0 B | | `dist/main.js` | 343 kB | -1 B (0%) | | `dist/openmrs-form-engine-lib.js` | 3.72 kB | 0 B |

compressed-size-action