-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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): Pin all data regardless of pagination #6346
Conversation
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 #6346 +/- ##
==========================================
+ Coverage 27.66% 28.31% +0.65%
==========================================
Files 2958 2982 +24
Lines 181566 185130 +3564
Branches 19732 20296 +564
==========================================
+ Hits 50225 52425 +2200
- Misses 130593 131920 +1327
- Partials 748 785 +37
☔ 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.
As far as I know, pinData
still needs to be paginated. Other than that, the fix works as expected.
✅ All Cypress E2E specs passed |
* master: fix(editor): Fix hard-coded parameter names for code editors (#6372) ci: Add support for Node.js 18 (#5793) ci: Update the release process for v1 release (#6366) refactor: Add deprecation notice for UM disabled mode (#6367) feat(editor): Make WF name a link on /executions (#6354) ci: Revert "Set up Turborepo cache for GitHub actions runner" (#6364) (no-changelog) fix(editor): Add secondary icon to menu items (#6351) fix(editor): Pin all data regardless of pagination (#6346) feat(Crypto Node): Add support for hash and hmac on binary data (#6359) ci: Set up Turborepo cache for GitHub actions runner (no-changelog) (#6335) # Conflicts: # packages/editor-ui/src/components/ParameterInput.vue
Got released with |
* fix(editor): pin all data * fix(editor): restore pinned data pagination
No description provided.