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

Conversation

@liweitian
Copy link
Contributor

Description

A url bug was introduced in recent Pr. When a skill bot is missing a luis/qna key, we can click on the error and navigate to settings page in Design page project tree. But the navigation url is incorrect. It passed current project id which should be root bot project Id.

Task Item

#minor

Screenshots

bug1

@coveralls
Copy link

coveralls commented Jan 18, 2021

Coverage Status

Coverage remained the same at 55.086% when pulling 64a13bf on bugfix1 into a7a78f8 on main.

@boydc2014 boydc2014 merged commit 66f6854 into main Jan 18, 2021
@boydc2014 boydc2014 deleted the bugfix1 branch January 18, 2021 11:47
lei9444 pushed a commit that referenced this pull request Jan 18, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
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
benbrown pushed a commit that referenced this pull request Jun 11, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
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