Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@zhixzhan
Copy link
Contributor

Description

When delete trigger from ProjectTree, update dialog should take projectId passed from projectTree, instead of current page skillId.

Task Item

Fixes #minor

Screenshots

image

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 55.081% when pulling 8872dbf on zhixzhan/trigger-delete-fix into 2d28769 on main.

@zhixzhan zhixzhan merged commit 6f797df into main Jan 18, 2021
@zhixzhan zhixzhan deleted the zhixzhan/trigger-delete-fix branch January 18, 2021 07:41
alanlong9278 added a commit that referenced this pull request Jan 25, 2021
* main:
  Update numberinput.dialog (#5575)
  fix hover display incorrect returntype (#5588)
  fix: Move persistence layer's delta computation into worker (#5563)
  fix: electron update error (#5573)
  fix: showing correct error message in local publish (#5509)
  feat: change source of packages from local feed to live npm/nuget feed (#5516)
  set max http header size to fix 431 (#5521)
  Updating to daily runtime for R12 development (#5529)
  fix: correctly generate l10n files when using zsh (#5555)
  chore: deprecate feature request issue template (#5378)
  fix: designPage navigation to settings Page url error (#5546)
  fix: luis\qna key missing in skill bot (#5545)
  delete trigger by projectId passed from projectTree (#5542)
benbrown pushed a commit to benbrown/BotFramework-Composer that referenced this pull request May 24, 2021
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants