From b95fcd73235fed85e0e644541738f7e1d138a4c4 Mon Sep 17 00:00:00 2001 From: Csaba Tuncsik Date: Fri, 12 May 2023 10:13:42 +0200 Subject: [PATCH] refactor(editor): Turn showMessage mixin to composable (#6081) * refactor(editor): move $getExecutionError from showMessages mixin to pushConnection (it is used there only) * refactor(editor): resolve showMessage mixin methods * fix(editor): use composable instead of mixin * fix(editor): resolve conflicts * fix(editor): replace clearAllStickyNotifications * fix(editor): replace confirmMessage * fix(editor): replace confirmMessage * fix(editor): replace confirmMessage * fix(editor): remove last confirmMessage usage * fix(editor): remove $prompt usage * fix(editor): remove $show methods * fix(editor): lint fix * fix(editor): lint fix * fix(editor): fixes after review --- packages/editor-ui/src/App.vue | 30 +-- .../src/components/ChangePasswordModal.vue | 19 +- .../src/components/CommunityPackageCard.vue | 7 +- .../CommunityPackageInstallModal.vue | 19 +- .../CommunityPackageManageConfirmModal.vue | 26 +- .../src/components/ContactPromptModal.vue | 10 +- .../editor-ui/src/components/CopyInput.vue | 11 +- .../src/components/CredentialCard.vue | 26 +- .../CredentialEdit/CredentialEdit.vue | 89 ++++--- .../CredentialEdit/CredentialSharing.ee.vue | 32 ++- .../src/components/DeleteUserModal.vue | 20 +- .../components/DuplicateWorkflowDialog.vue | 17 +- .../src/components/Error/NodeErrorView.vue | 12 +- .../src/components/ExecutionsList.vue | 48 ++-- .../ExecutionsView/ExecutionCard.vue | 3 +- .../ExecutionsView/ExecutionPreview.vue | 25 +- .../ExecutionsView/ExecutionsList.vue | 58 +++-- .../src/components/ImportCurlModal.vue | 21 +- .../src/components/ImportParameter.vue | 5 +- .../src/components/InlineNameEdit.vue | 13 +- .../src/components/InviteUsersModal.vue | 19 +- .../components/MainHeader/WorkflowDetails.vue | 31 ++- .../editor-ui/src/components/MainSidebar.vue | 16 +- .../src/components/NodeCredentials.vue | 11 +- .../src/components/NodeDetailsView.vue | 14 +- .../src/components/NodeExecuteButton.vue | 21 +- .../editor-ui/src/components/NodeWebhooks.vue | 11 +- .../components/OnboardingCallSignupModal.vue | 15 +- .../editor-ui/src/components/PageAlert.vue | 20 +- .../src/components/ParameterInput.vue | 9 +- .../src/components/ParameterInputFull.vue | 13 +- .../src/components/ParameterInputWrapper.vue | 5 +- .../src/components/PersonalizationModal.vue | 15 +- packages/editor-ui/src/components/RunData.vue | 11 +- .../src/components/RunDataJsonActions.vue | 12 +- .../EventDestinationCard.ee.vue | 29 ++- .../EventDestinationSettingsModal.ee.vue | 54 ++-- .../editor-ui/src/components/TagsDropdown.vue | 13 +- .../components/TagsManager/TagsManager.vue | 21 +- .../editor-ui/src/components/TriggerPanel.vue | 21 +- .../editor-ui/src/components/ValueSurvey.vue | 10 +- .../src/components/WorkflowActivator.vue | 14 +- .../editor-ui/src/components/WorkflowCard.vue | 33 ++- .../src/components/WorkflowPreview.vue | 17 +- .../src/components/WorkflowSettings.vue | 25 +- .../src/components/WorkflowShareModal.ee.vue | 55 ++-- .../__tests__/ExecutionsList.test.ts | 3 +- .../layouts/ResourcesListLayout.vue | 5 +- packages/editor-ui/src/composables/index.ts | 7 +- .../editor-ui/src/composables/useToast.ts | 26 +- packages/editor-ui/src/constants.ts | 2 +- .../editor-ui/src/mixins/genericHelpers.ts | 13 +- packages/editor-ui/src/mixins/newVersions.ts | 13 +- packages/editor-ui/src/mixins/pinData.ts | 10 +- .../editor-ui/src/mixins/pushConnection.ts | 59 +++-- packages/editor-ui/src/mixins/showMessage.ts | 235 ------------------ .../editor-ui/src/mixins/workflowActivate.ts | 15 +- .../editor-ui/src/mixins/workflowHelpers.ts | 30 ++- packages/editor-ui/src/mixins/workflowRun.ts | 9 +- .../src/plugins/i18n/locales/en.json | 4 +- .../src/views/ChangePasswordView.vue | 21 +- .../editor-ui/src/views/CredentialsView.vue | 18 +- .../src/views/ForgotMyPasswordView.vue | 15 +- packages/editor-ui/src/views/NodeView.vue | 124 +++++---- .../editor-ui/src/views/SettingsApiView.vue | 33 ++- .../src/views/SettingsCommunityNodesView.vue | 11 +- .../editor-ui/src/views/SettingsLdapView.vue | 50 ++-- .../src/views/SettingsPersonalView.vue | 15 +- .../editor-ui/src/views/SettingsUsersView.vue | 17 +- packages/editor-ui/src/views/SetupView.vue | 38 +-- packages/editor-ui/src/views/SigninView.vue | 13 +- packages/editor-ui/src/views/SignoutView.vue | 13 +- packages/editor-ui/src/views/SignupView.vue | 19 +- .../src/views/TemplatesSearchView.vue | 10 +- .../editor-ui/src/views/WorkflowsView.vue | 18 +- 75 files changed, 990 insertions(+), 862 deletions(-) delete mode 100644 packages/editor-ui/src/mixins/showMessage.ts diff --git a/packages/editor-ui/src/App.vue b/packages/editor-ui/src/App.vue index 7bbd5c668a..14aec36ec8 100644 --- a/packages/editor-ui/src/App.vue +++ b/packages/editor-ui/src/App.vue @@ -27,29 +27,28 @@