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

fix: add disk names#2274

Merged
cwhitten merged 6 commits intomicrosoft:masterfrom
liweitian:showDiskName
Mar 25, 2020
Merged

fix: add disk names#2274
cwhitten merged 6 commits intomicrosoft:masterfrom
liweitian:showDiskName

Conversation

@liweitian
Copy link
Contributor

@liweitian liweitian commented Mar 16, 2020

Description

  1. The bug only exists on windows because of hard disk partition.
  2. The parent of root disk will be This PC. Go to 'This PC', it will show all existing disk names.
  3. Note that this is a corner case, Path module does not recognize 'This PC' as a valid path.

Task Item

Closes #2257

Screenshots

aaaaa

@liweitian liweitian force-pushed the showDiskName branch 2 times, most recently from 97918f5 to 6dd0ff1 Compare March 17, 2020 14:32
@boydc2014
Copy link
Contributor

Looks way better now.

boydc2014
boydc2014 previously approved these changes Mar 18, 2020
@boydc2014 boydc2014 added the Approved to merge approved, waiting to be merged label Mar 18, 2020
yeze322
yeze322 previously approved these changes Mar 19, 2020
@cwhitten
Copy link
Member

@liweitian please address conflicts

@liweitian
Copy link
Contributor Author

liweitian commented Mar 23, 2020

@liweitian please address conflicts

it is done.

Copy link
Member

@cwhitten cwhitten left a comment

Choose a reason for hiding this comment

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

yarn lint failing with:

| /home/runner/work/BotFramework-Composer/BotFramework-Composer/Composer/packages/client/src/CreationFlow/DefineConversation/index.tsx
| 4:1 error There should be at least one empty line between import groups import/order

@cwhitten cwhitten dismissed stale reviews from a-b-r-o-w-n and themself March 25, 2020 15:43

stale

@cwhitten cwhitten merged commit 277299c into microsoft:master Mar 25, 2020
cwhitten added a commit that referenced this pull request Mar 25, 2020
cwhitten added a commit that referenced this pull request Mar 25, 2020
This was referenced Mar 29, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* add disk names

* add disk names

* remove console

* handle comments

* do not allow creating bot in path 'this PC'

* handle comments
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

Approved to merge approved, waiting to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants