-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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(editor): Fix typing $
in inline expression field reloading node parameters form
#6374
fix(editor): Fix typing $
in inline expression field reloading node parameters form
#6374
Conversation
… parameters form
Great PR! Please pay attention to the following items before merging: Files matching
Files matching
Files matching
Make sure to check off this list before asking for review. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #6374 +/- ##
==========================================
- Coverage 28.32% 28.26% -0.07%
==========================================
Files 2982 2982
Lines 185204 185215 +11
Branches 20304 20303 -1
==========================================
- Hits 52465 52344 -121
- Misses 131952 132084 +132
Partials 787 787
☔ View full report in Codecov by Sentry. |
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, fixes issue
|
1 flaky tests on run #1092 ↗︎Details:
|
Test | Artifacts | |
---|---|---|
Workflows > should delete all the workflows |
Output
Screenshots
Video
|
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.
✅ All Cypress E2E specs passed |
… parameters form (#6374) * fix(editor): Fix typing `$` in inline expression field reloading node parameters form * ⚡ Setting resource mapper empty field values to empty strings
* master: fix(editor): Handle null object on saving user settings (no-changelog) (#6341) docs: Fix trigger node link for Lonescale (#6376) ci: Improve Test-worklows messages (#6290) refactor: Upgrade to TypeScript 5.1 (no-changelog) (#6373) fix(editor): Fix typing `$` in inline expression field reloading node parameters form (#6374) 🚀 Release 0.231.0 (#6344) # Conflicts: # pnpm-lock.yaml
… parameters form (#6374) * fix(editor): Fix typing `$` in inline expression field reloading node parameters form * ⚡ Setting resource mapper empty field values to empty strings
Got released with |
In #6101 we implemented dependent parameters support for the Resource Locator Component which reloaded RLC if any of the node parameters it depends on is updated. However, this solution was causing a reload every time
watch
is triggered without checking if the value actually changed.This PR fixes that and few other minor things:
ParameterInput
:void
-->await
,ParameterInputList
ResourceMapper
component optional