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(editor): Fix save modal appearing post duplication of a workflow #5247

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

freyamade
Copy link
Contributor

The versionId in the store was being updated before any checking of "should this open in a new window" was done. This led to the versionId of the new duplicated workflow being used in the tab of the original workflow, leading to the save modal popping up erroneusly

@freyamade freyamade changed the title 🐛 - fixed post duplication save issue fix(ui) - Fix save modal appearing post duplication of a workflow Jan 25, 2023
@freyamade freyamade changed the title fix(ui) - Fix save modal appearing post duplication of a workflow fix(editor) - Fix save modal appearing post duplication of a workflow Jan 25, 2023
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Jan 25, 2023
@freyamade freyamade changed the title fix(editor) - Fix save modal appearing post duplication of a workflow fix(editor): Fix save modal appearing post duplication of a workflow Jan 25, 2023
@freyamade freyamade merged commit c711c53 into master Jan 25, 2023
@freyamade freyamade deleted the pay-68-bug-overwrite-changes-modal-triggered branch January 25, 2023 15:09
@n8n-assistant n8n-assistant bot added the Upcoming Release Will be part of the upcoming release label Jan 25, 2023
MiloradFilipovic added a commit that referenced this pull request Jan 25, 2023
* master:
  fix(editor): Fix save modal appearing after duplicating a workflow (#5247)
  feat(editor): Adjust Google sign-in button to adhere to the guidelines (#5248)
  fix(HelpScout Node): Fix tag search not working when getting all conversations (#5239)
  fix(editor): Do not request workflow data twice when opening a workflow (#5246)
  fix(editor): Fix the element-ui imports in SettingsLdapView (no-changelog) (#5245)
  fix(core): Handle missing binary metadata in download urls (#5242)
  ci: Simplify DB truncate in tests (no-changelog) (#5243)
  feat(core): Add LDAP support (#3835)
  fix(core): Upsert credentials and workflows in the import:* commands (#5231)
  feat(Jira Software Node): Use resource locator component (#5090)
  ci: Publish n8n docker images to GHCR (#5213)
  fix(Google Drive Node): Use the correct mimetype on converted downloads (#5240)
  fix(editor): Prevent workflow execution list infinite no network error (#5230)
  fix: Extension being too eager and making calls when it shouldn't (#5232)
  feat(Send Email Node): Overhaul
  refactor(core): Add support for implicit schema in postgres migrations (#5233)

# Conflicts:
#	packages/editor-ui/src/components/CredentialEdit/CredentialConfig.vue
@janober
Copy link
Member

janober commented Jan 27, 2023

Got released with [email protected]

@janober janober removed the Upcoming Release Will be part of the upcoming release label Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants