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

Conversation

@alanlong9278
Copy link
Contributor

Description

Refactor createOptions modal for abs depends on new design. https://www.figma.com/file/DxN7GOTds0xII7Hp1zeYjtuW/ABS-main?node-id=1888%3A963
image

Old scenario

  • alias data have value
    • jump to bot design page straight
  • alias data empty
    • open create new bot modal

New scenario

  • alias data have value
    • open choice modal to choose if open exist bot or create new bot

abs

  • alias data empty
    • open create new bot modal

Task Item

#minor

Screenshots

@coveralls
Copy link

coveralls commented Mar 23, 2021

Coverage Status

Coverage increased (+0.002%) to 52.733% when pulling 352b1af on julong/abs-createOptions-modal into 4dd99f4 on main.

tonyanziano
tonyanziano previously approved these changes Mar 24, 2021
Copy link
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

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

It looks pretty good to me!

@luhan2017 luhan2017 merged commit 4a08f6a into main Mar 24, 2021
@luhan2017 luhan2017 deleted the julong/abs-createOptions-modal branch March 24, 2021 06:38
@cwhitten cwhitten mentioned this pull request May 20, 2021
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* refactor create option modal

* test

* lint

* rename isOpen to isOpenOptionsModal

Co-authored-by: Lu Han <32191031+luhan2017@users.noreply.github.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.

5 participants