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

Fix copy paste of widget value #284

Merged
merged 5 commits into from
Aug 3, 2024
Merged

Fix copy paste of widget value #284

merged 5 commits into from
Aug 3, 2024

Conversation

huchenlei
Copy link
Member

This PR refactors the workflow validation to load of graph as in many other places, such as on paste, the parse is just to check whether the string is a JSON.

Previously copy of widget value would trigger an zod validation error alert. This PR adds a playwright test to reproduce that.

@huchenlei huchenlei added the New Browser Test Expectations New browser test screenshot should be set by github action label Aug 3, 2024
@huchenlei huchenlei added New Browser Test Expectations New browser test screenshot should be set by github action and removed New Browser Test Expectations New browser test screenshot should be set by github action labels Aug 3, 2024
@huchenlei huchenlei merged commit 0cf5e64 into main Aug 3, 2024
3 checks passed
@huchenlei huchenlei deleted the fix_copy branch August 3, 2024 18:22
huchenlei added a commit that referenced this pull request Aug 3, 2024
* Fix canvas not init issue (#283)

* Fix copy paste of widget value (#284)

* Fix copy paste of widget value

* Fix ui tests

* Allow undefined group font size

* Update test expectations [skip ci]

* nit

---------

Co-authored-by: github-actions <github-actions@github.com>

* 1.2.9 (#285)

* WIP

* Add refresh button

* Add context menu

* nit

* Add selection mode

* Editable text

* Fix relative path

* implement node delete

* Dynamic menu items

* Fix refresh

* Better dynamic handling of menu items

* Disable rename / delete for root

* Add workflow download

* Auto select file name

* Create workflow

* Generate non-dup name

* Fix folder name

* Rename workflwoStore to userFileStore

* load workflow when leaf node selected

* Extract common report error logic

* Basic workflows tab test

* Auto expand

* Add test on add/remove workflow

---------

Co-authored-by: github-actions <github-actions@github.com>
huchenlei added a commit that referenced this pull request Aug 4, 2024
* Fix canvas not init issue (#283)

* Fix copy paste of widget value (#284)

* Fix copy paste of widget value

* Fix ui tests

* Allow undefined group font size

* Update test expectations [skip ci]

* nit

---------

Co-authored-by: github-actions <github-actions@github.com>

* 1.2.9 (#285)

* WIP

* Add refresh button

* Add context menu

* nit

* Add selection mode

* Editable text

* Fix relative path

* implement node delete

* Dynamic menu items

* Fix refresh

* Better dynamic handling of menu items

* Disable rename / delete for root

* Add workflow download

* Auto select file name

* Create workflow

* Generate non-dup name

* Fix folder name

* Rename workflwoStore to userFileStore

* load workflow when leaf node selected

* Extract common report error logic

* Basic workflows tab test

* Auto expand

* Add test on add/remove workflow

---------

Co-authored-by: github-actions <github-actions@github.com>
huchenlei added a commit that referenced this pull request Aug 5, 2024
* Fix canvas not init issue (#283)

* Fix copy paste of widget value (#284)

* Fix copy paste of widget value

* Fix ui tests

* Allow undefined group font size

* Update test expectations [skip ci]

* nit

---------

Co-authored-by: github-actions <github-actions@github.com>

* 1.2.9 (#285)

* WIP

* Add refresh button

* Add context menu

* nit

* Add selection mode

* Editable text

* Fix relative path

* implement node delete

* Dynamic menu items

* Fix refresh

* Better dynamic handling of menu items

* Disable rename / delete for root

* Add workflow download

* Auto select file name

* Create workflow

* Generate non-dup name

* Fix folder name

* Rename workflwoStore to userFileStore

* load workflow when leaf node selected

* Extract common report error logic

* Basic workflows tab test

* Auto expand

* Add test on add/remove workflow

---------

Co-authored-by: github-actions <github-actions@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Browser Test Expectations New browser test screenshot should be set by github action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant