-
Notifications
You must be signed in to change notification settings - Fork 54
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
vdk-jupyter: Jupyter UI change the strategy on storing the data from user input #1624
vdk-jupyter: Jupyter UI change the strategy on storing the data from user input #1624
Conversation
for more information, see https://pre-commit.ci
I got it, this change aims to shift the session-scoped data to conversation-scoped one. |
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.
LGTM
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/components/CreateJob.tsx
Outdated
Show resolved
Hide resolved
for more information, see https://pre-commit.ci
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/components/CreateJob.tsx
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/dataClasses/jobData.ts
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/handlers.py
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/vdk_ui.py
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/vdk_ui.py
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/handlers.py
Outdated
Show resolved
Hide resolved
...cts/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/dict_object.py
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/handlers.py
Outdated
Show resolved
Hide resolved
b481a88
to
8f7090a
Compare
for more information, see https://pre-commit.ci
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.
LVGTM
...k-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/ui_job_properties.py
Outdated
Show resolved
Hide resolved
...k-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/ui_job_properties.py
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/jobData.ts
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/jobData.ts
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/jobData.ts
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/jobData.ts
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/README.md
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/README.md
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/README.md
Outdated
Show resolved
Hide resolved
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/src/components/RunJob.tsx
Outdated
Show resolved
Hide resolved
for more information, see https://pre-commit.ci
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.
LGTM
projects/vdk-plugins/vdk-jupyter/vdk-jupyterlab-extension/vdk-jupyterlab-extension/vdk_ui.py
Outdated
Show resolved
Hide resolved
3ac0b8b
to
c6ace5e
Compare
for more information, see https://pre-commit.ci
What:
Changed the strategy on storing the data from user input for each vdk operation.
Why:
It is linked to the #1484
It is also linked to the problem - saving SessionStorage data is really error-prone. SessionStorage keys are pure strings and even a single typo can create many unwanted behaviour of the project.
With this solution we remove the sessionStorage saved data and save them in global variables which are dataclasses that describe the input data from a user for specific vdk operation.
Note: If this solution is reviewed better than the current solution with the SessionStorage, the #1621 PR will be removed
Signed-off-by: Duygu Hasan hduygu@vmware.com