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
Fix MSP_BOARD_INFO accumulation and localStorage Quota Exceeded #2937
Conversation
This comment has been minimized.
This comment has been minimized.
f8a295b
to
095efbb
Compare
This comment has been minimized.
This comment has been minimized.
095efbb
to
4b0fe7d
Compare
This comment has been minimized.
This comment has been minimized.
AUTOMERGE: (FAIL)
|
4b0fe7d
to
8962c82
Compare
This comment has been minimized.
This comment has been minimized.
c21b9d5
to
bae0dcb
Compare
This comment has been minimized.
This comment has been minimized.
bae0dcb
to
8e247a0
Compare
This comment has been minimized.
This comment has been minimized.
8e247a0
to
925ff52
Compare
This comment has been minimized.
This comment has been minimized.
What happens with old values stored in the local storage? They will remain there until the user deletes the folder in the computer? |
Pushed a new update to remove old keys |
d9236de
to
08803d2
Compare
This comment has been minimized.
This comment has been minimized.
Cleanup LocalStorage
08803d2
to
b51a7b2
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Do you want to test this code? Here you have an automated build: |
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.
The cleanup is very simple, maybe we need to store some "version" in the localStorage, and act according to the change of version. But for the moment, I think is enough with your code. I suppose this is a exceptional situation and we can remove the cleaning in one or two versions.
Fixes: #2938
FC.CONFIG.boardName
parameter accumulating previous values.Loader
function in atry catch error
statement.Jenkins
+ firmwarecache
fromlocalStorage
to newSessionStorage
to fixExceeded Quota
error.