get localstorage support (#2190)

[dinhlongviolin1]

Resolve #2190

localStorage.setItem and localStorage.removeItem can be used directly on the browser console to test out the feature

from taipy.gui import Gui, get_local_storage


def button_click(state):
    print(get_local_storage(state, "TaipyClientId")) # would return str | None
    print(get_local_storage(state, "TaipyClientId", "hey")) # would return dict

Gui("<|Click me|button|on_action=button_click|>").run()

[github-actions]

Coverage report for ./frontend/taipy

Caution

Coverage does not meet threshold
Statements coverage not met for global: expected >=80%, but got 51.87376725838264%

St.❔	Category	Percentage	Covered / Total
🔴	Statements	51.87%	263/507
🔴	Branches	21.61%	67/310
🔴	Functions	14.66%	17/116
🔴	Lines	53.19%	242/455

Test suite run success

7 tests passing in 1 suite.

Report generated by 🧪jest coverage report action from 5814635

[github-actions]

Coverage report for ./frontend/taipy-gui

Caution

Coverage does not meet threshold
Branches coverage not met for global: expected >=80%, but got 69.25624811803674%

St.❔	Category	Percentage	Covered / Total
🟢	Statements	87.77% (-0.07% 🔻)	3408/3883
🟡	Branches	69.26% (-0.04% 🔻)	2300/3321
🟢	Functions	83.42% (-0.22% 🔻)	629/754
🟢	Lines	88.24% (-0.06% 🔻)	3152/3572

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🟢	... / taipyReducers.ts	89.49% (-0.97% 🔻)	81.22% (-0.83% 🔻)	83.64% (-3.16% 🔻)	89.92% (-0.91% 🔻)

Test suite run success

681 tests passing in 47 suites.

Report generated by 🧪jest coverage report action from a98cddf

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
19506	16995	87%	0%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
taipy/gui/init.py	100%	🟢
taipy/gui/data/data_scope.py	94%	🟢
taipy/gui/gui.py	79%	🟢
taipy/gui/gui_actions.py	64%	🟢
taipy/gui/types.py	96%	🟢
TOTAL	87%	🟢

updated for commit: 5814635 by action🐍

[FredLL-Avaiga]

Not sure about this local storage API redéfinition in the browser
Way to invasive for my taste
And I'm not sure a security audit would appreciate

[FabienLelaquais]

I would like to improve the doc a little bit before you actually merge.
Let's do that together if that works for you.

[FabienLelaquais]

Line skip here.

Can this be called anytime in the app life cycle?

I suspect we should add more details on invoking this API (with the complexity of the keys type).

[dinhlongviolin1]

i have not tested every case but it is very early on, only after the state id is initialized.

[FabienLelaquais]

Line skip.

Explanations are pretty weak. Let's spend some time together on this one.

[dinhlongviolin1]

i am open to any proposal to the doc

[github-actions]

This PR has been labelled as "🥶Waiting for contributor" because it has been inactive for more than 14 days. If you would like to continue working on this PR, then please add new commit or another comment, otherwise this PR will be closed in 14 days. For more information please refer to the contributing guidelines.

[github-actions]

This PR has been closed because it has been marked as "🥶Waiting for contributor" for more than 14 days with no activity.