Skip to content
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

Optimize the size of the resource #211

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

hueifeng
Copy link
Member

Before optimization:

> [email protected] build
> ng build

✔ Browser application bundle generation complete.
✔ Copying assets complete.
✔ Index html generation complete.

Initial Chunk Files              | Names         |      Size
vendor.js                        | vendor        |   3.61 MB
polyfills-es5.js                 | polyfills-es5 | 608.47 kB
styles.css                       | styles        | 203.20 kB
polyfills.js                     | polyfills     | 171.98 kB
main.js                          | main          |  13.03 kB
runtime.js                       | runtime       |  12.92 kB

                                 | Initial Total |   4.59 MB

Lazy Chunk Files                 | Names         |      Size
src_app_pages_pages_module_ts.js | -             |   2.88 MB

After the optimization:

PS I:\source\dashboard\web> npm run build

> [email protected] build
> ng build

✔ Browser application bundle generation complete.
✔ Copying assets complete.
✔ Index html generation complete.

Initial Chunk Files                                                         | Names                  |      Size
dapr.angular.js                                                             | dapr.angular           |   1.61 MB
styles.css                                                                  | styles                 | 203.20 kB
dapr.core-js.js                                                             | dapr.core-js           |  82.05 kB
dapr.rxjs.js                                                                | dapr.rxjs              |  80.79 kB
vendor.js                                                                   | vendor                 |  61.18 kB
dapr.web-animations-js.js                                                   | dapr.web-animations-js |  46.04 kB
dapr.zone.js.js                                                             | dapr.zone.js           |  44.66 kB
polyfills-es5.js                                                            | polyfills-es5          |   9.37 kB
main.js                                                                     | main                   |   3.64 kB
runtime.js                                                                  | runtime                |   3.10 kB
polyfills.js                                                                | polyfills              |   2.10 kB

                                                                            | Initial Total          |   2.13 MB

Lazy Chunk Files                                                            | Names                  |      Size
dapr.moment.js                                                              | dapr.moment            | 286.78 kB
src_app_pages_pages_module_ts-node_modules_moment_locale_sync_recursive_.js | -                      | 150.20 kB
dapr.js-yaml.js                                                             | dapr.js-yaml           |  38.66 kB

shubham1172 and others added 2 commits June 18, 2022 15:35
Signed-off-by: Shubham Sharma <[email protected]>
Signed-off-by: HueiFeng <[email protected]>
Signed-off-by: HueiFeng <[email protected]>
@codecov
Copy link

codecov bot commented Jun 21, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (298a16a) 27.56% compared to head (45e4559) 27.56%.

❗ Current head 45e4559 differs from pull request most recent head 407529a. Consider uploading reports for the commit 407529a to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #211   +/-   ##
=======================================
  Coverage   27.56%   27.56%           
=======================================
  Files           7        7           
  Lines         798      798           
=======================================
  Hits          220      220           
  Misses        554      554           
  Partials       24       24           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# Conflicts:
#	web/package-lock.json
@yaron2
Copy link
Member

yaron2 commented Feb 14, 2024

@hueifeng can you please resolve the conflict? we should be able to merge then.

# Conflicts:
#	web/package-lock.json

Signed-off-by: HueiFeng <[email protected]>
Signed-off-by: Huei Feng <[email protected]>
Signed-off-by: Huei Feng <[email protected]>
Signed-off-by: hueifeng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants