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

Conversation

@liweitian
Copy link
Contributor

@liweitian liweitian commented Apr 7, 2021

Description

root cause: target path is not updated if it is focusing on the input field.

Task Item

Closes #6225

Screenshots

dddd

filepath

@coveralls
Copy link

coveralls commented Apr 7, 2021

Coverage Status

Coverage decreased (-0.02%) to 51.148% when pulling e4f7ae5 on fileSelectorBug into ad43489 on main.

Copy link
Contributor

@hatpick hatpick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think here is a good opportunity to test the path and if it's invalid show an error or something, and prevent moving forward

@cwhitten cwhitten added this to the R13 milestone Apr 7, 2021
@liweitian liweitian requested a review from benbrown as a code owner April 12, 2021 09:01
@liweitian
Copy link
Contributor Author

liweitian commented Apr 12, 2021

I think here is a good opportunity to test the path and if it's invalid show an error or something, and prevent moving forward

@hatpick path validation added

@cwhitten cwhitten merged commit ce10c9e into main Apr 12, 2021
@cwhitten cwhitten deleted the fileSelectorBug branch April 12, 2021 22:47
alanlong9278 added a commit that referenced this pull request Apr 13, 2021
* qika/skill:
  lint
  callers
  comments
  Fix #6731: use resourceGroup name from provisioning profile if one exists (#6872)
  fix: file selector bug (#6733)
  feat: Move Debug Panel outside the main content container (#6870)
  - Fixing add new skill by fetch correct templates on addSkill modal (#6800)
  update bf-dialog to latest rc, adjust syntax (#6849)
  feat: Update home feed and add content (#6829)
  add create publish profile dialog
@cwhitten cwhitten mentioned this pull request May 20, 2021
benbrown pushed a commit to benbrown/BotFramework-Composer that referenced this pull request May 24, 2021
* qika/skill:
  lint
  callers
  comments
  Fix microsoft#6731: use resourceGroup name from provisioning profile if one exists (microsoft#6872)
  fix: file selector bug (microsoft#6733)
  feat: Move Debug Panel outside the main content container (microsoft#6870)
  - Fixing add new skill by fetch correct templates on addSkill modal (microsoft#6800)
  update bf-dialog to latest rc, adjust syntax (microsoft#6849)
  feat: Update home feed and add content (microsoft#6829)
  add create publish profile dialog
benbrown pushed a commit that referenced this pull request Jun 11, 2021
* qika/skill:
  lint
  callers
  comments
  Fix #6731: use resourceGroup name from provisioning profile if one exists (#6872)
  fix: file selector bug (#6733)
  feat: Move Debug Panel outside the main content container (#6870)
  - Fixing add new skill by fetch correct templates on addSkill modal (#6800)
  update bf-dialog to latest rc, adjust syntax (#6849)
  feat: Update home feed and add content (#6829)
  add create publish profile dialog
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* fix file selector bug

* validate path

* add path validation

* add comment

Co-authored-by: Soroush <hatpick@gmail.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.

bot is created in wrong folder

5 participants