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
perf(http): reduce data transfer when using HTTP caching #52347
Conversation
How about using tuples? |
I went with a similar approach that is used for other things like hydration. Also with tuples, it feel like it would be harder to visually read what is written in the cache. |
43563ed
to
c26d496
Compare
c26d496
to
66ad393
Compare
This commit reduces the property size in the http transfer cache to reduce the page payload. Before ```html <script id="ng-state" type="application/json"> { "4155228514": { "body": "....", "headers": {}, "status": 200, "statusText": "OK", "url": "http://foo.com/assets/media.json", "responseType": "json" }, } </script> ``` Now ```html <script id="ng-state" type="application/json"> { "4155228514": { "b": "....", "h": {}, "s": 200, "st": "OK", "u": "http://foo.com/assets/media.json", "rt": "json" }, } </script> ```
66ad393
to
02acbb4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! 👍
This PR was merged into the repository by commit c5e30f1. |
This commit reduces the property size in the http transfer cache to reduce the page payload. Before ```html <script id="ng-state" type="application/json"> { "4155228514": { "body": "....", "headers": {}, "status": 200, "statusText": "OK", "url": "http://foo.com/assets/media.json", "responseType": "json" }, } </script> ``` Now ```html <script id="ng-state" type="application/json"> { "4155228514": { "b": "....", "h": {}, "s": 200, "st": "OK", "u": "http://foo.com/assets/media.json", "rt": "json" }, } </script> ``` PR Close #52347
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This commit reduces the property size in the http transfer cache to reduce the page payload. Before ```html <script id="ng-state" type="application/json"> { "4155228514": { "body": "....", "headers": {}, "status": 200, "statusText": "OK", "url": "http://foo.com/assets/media.json", "responseType": "json" }, } </script> ``` Now ```html <script id="ng-state" type="application/json"> { "4155228514": { "b": "....", "h": {}, "s": 200, "st": "OK", "u": "http://foo.com/assets/media.json", "rt": "json" }, } </script> ``` PR Close angular#52347
This commit reduces the property size in the http transfer cache to reduce the page payload. Before ```html <script id="ng-state" type="application/json"> { "4155228514": { "body": "....", "headers": {}, "status": 200, "statusText": "OK", "url": "http://foo.com/assets/media.json", "responseType": "json" }, } </script> ``` Now ```html <script id="ng-state" type="application/json"> { "4155228514": { "b": "....", "h": {}, "s": 200, "st": "OK", "u": "http://foo.com/assets/media.json", "rt": "json" }, } </script> ``` PR Close angular#52347
This commit reduces the property size in the http transfer cache to reduce the page payload.
Before
Now