This repository was archived by the owner on Jul 9, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 375
feat: updated "Get started" flow available on every page #6365
Merged
Merged
Changes from 54 commits
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
ff774ff
Enable the get started widget to intelligently determine if LUIS or Q…
benbrown 341485f
define nextsteps as an array that gets rendered
benbrown 7cf855a
adjsut formating
benbrown 1f17f6f
Merge branch 'main' into benbrown/getsmarted
benbrown b092013
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown 38e1f42
move get started to top blue bar
benbrown a96a102
Add placeholder mvp luis keys modal
benbrown a9cacd9
add placeholder mvp qna key modal
benbrown 117b497
introduce ProvisionHandoff component, to display customized instructi…
benbrown 7b03f51
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown 364016c
luis widget
benbrown 765d843
Merge branch 'main' into benbrown/getsmarted
benbrown 2105dab
updated styles and content for get started
benbrown fd702c1
clean up code
benbrown 4f03203
Merge branch 'benbrown/getsmarted' of https://github.com/microsoft/Bo…
benbrown 41d459b
add handoff step to luis widget
benbrown 2ded7c2
add support for qnamaker provisioning
benbrown fbe62f9
merge main
benbrown 1d415c1
remove unused css
benbrown 3b69ced
Merge branch 'main' into benbrown/getsmarted
benbrown 709f3de
clean up unused vars
benbrown b19dc34
Merge branch 'main' into benbrown/getsmarted
benbrown 8e984c2
add final confirmation step to luis
benbrown 5c95489
updated ux for luis
benbrown 7b3a497
fix button disabled state
benbrown d2127d7
Merge branch 'benbrown/getsmarted' of https://github.com/microsoft/Bo…
benbrown ff12ee4
add generate request option to provisioning flow
benbrown 53a3bd1
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown 8414ddd
Merge branch 'benbrown/getsmarted' into benbrown/genreq
benbrown 1ef07d8
ux fixes for luis
benbrown 56a35ac
ux fixes for qna
benbrown f25a30a
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown 0ca0e8a
magic teaching bubbs
benbrown 26ca3bf
cleanup
benbrown 592a7d7
fix weird issue
benbrown 91762e7
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown f10e854
Wrapping up UI for provisioning hand off modal
pavolumMsft d4c1b02
Merge branch 'benbrown/getsmarted' of https://github.com/microsoft/Bo…
pavolumMsft 6cfd483
Merge branch 'benbrown/getsmarted' of https://github.com/microsoft/Bo…
benbrown 4744a6c
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown 95a05db
revised links and buttons in get started
benbrown 41a6aaa
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
benbrown d63a92a
remove unused styles
benbrown 3b396eb
improve behavior of teaching bubbles during creation flow
benbrown 8c9cdbd
highglight start bot when no steps are left
benbrown 9143465
Merge branch 'main' into benbrown/getsmarted
benbrown 4d16786
nav to page when opening modal
benbrown 9e122ab
remove delete bot from settings page (still in toolbar)
benbrown 338f7c4
make pane scrollable
benbrown 4c80ff7
delete unused ref
benbrown 29cf767
address issue with type mismatch in qna
benbrown 0469da7
fix mock
benbrown 966701d
fix test
benbrown 5465e1a
Merge branch 'main' into benbrown/getsmarted
benbrown e426691
Merge branch 'main' into benbrown/getsmarted
benbrown 7b97051
update syntax for accessing lufiles and qnafiles
benbrown 93a304d
address pull request feedback
benbrown 7c6ab48
Merge branch 'main' into benbrown/getsmarted
benbrown 78f0ae4
locale
benbrown a6d8f10
Merge branch 'benbrown/getsmarted' of https://github.com/microsoft/Bo…
benbrown 7eb5f63
Merge branch 'main' into benbrown/getsmarted
benbrown 49a6efb
Merge branch 'main' into benbrown/getsmarted
beyackle 1571926
Merge branch 'main' into benbrown/getsmarted
cwhitten 860006d
Merge branch 'main' into benbrown/getsmarted
beyackle cb7b995
Merge branch 'main' into benbrown/getsmarted
beyackle b11c5c0
Merge branch 'benbrown/genreq' into benbrown/getsmarted
benbrown fb995de
fix merge conflict
benbrown e484f5a
Merge branch 'main' into benbrown/getsmarted
beyackle dcc79e5
Merge branch 'main' into benbrown/getsmarted
benbrown 45688d6
Merge branch 'main' into benbrown/getsmarted
benbrown 08ee9a5
Merge branch 'main' into benbrown/getsmarted
benbrown File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
119 changes: 119 additions & 0 deletions
119
Composer/packages/client/src/components/GetStarted/GetStartedLearn.tsx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,119 @@ | ||
| // Copyright (c) Microsoft Corporation. | ||
| // Licensed under the MIT License. | ||
|
|
||
| /** @jsx jsx */ | ||
| import { jsx } from '@emotion/core'; | ||
| import React from 'react'; | ||
| import formatMessage from 'format-message'; | ||
| import { Link } from 'office-ui-fabric-react/lib/Link'; | ||
| import { ScrollablePane } from 'office-ui-fabric-react/lib/ScrollablePane'; | ||
|
|
||
| import TelemetryClient from '../../telemetry/TelemetryClient'; | ||
|
|
||
| import { h3Style, ulStyle, liStyle } from './styles'; | ||
|
|
||
| const linkToEmulator = 'https://aka.ms/composer-getstarted-emulator'; | ||
| const linkToQNA = 'https://aka.ms/composer-getstarted-qnamaker'; | ||
| const linkToPackages = 'https://aka.ms/composer-getstarted-packages'; | ||
| const linkToImport = 'https://aka.ms/composer-getstarted-importpublishing'; | ||
| const linkToPVA = 'https://aka.ms/composer-getstarted-pva'; | ||
| const linkToMigrate = 'https://aka.ms/composer-getstarted-migrate'; | ||
| const linkToFeatureRequest = 'https://aka.ms/composer-getstarted-featurerequest'; | ||
|
|
||
| const linkToAdaptiveExpressions = 'https://aka.ms/composer-getstarted-adaptiveexpressions'; | ||
| const linkToGetStarted = 'https://aka.ms/composer-getstarted-getstarted'; | ||
| const linkToCreateFirstBot = 'https://aka.ms/composer-getstarted-quickstart'; | ||
| const linkToTutorials = 'https://aka.ms/composer-getstarted-tutorials'; | ||
| const linkToLGFileFormat = 'https://aka.ms/composer-getstarted-lgfiles'; | ||
| const linkToLUFileFormat = 'https://aka.ms/composer-getstarted-lufiles'; | ||
| const linkToPreBuiltExpressions = 'https://aka.ms/composer-getstarted-prebuilt'; | ||
|
|
||
| const linkClick = (event) => { | ||
| TelemetryClient.track('GettingStartedLinkClicked', { method: 'link', url: event.target.href }); | ||
| }; | ||
|
|
||
| export const GetStartedLearn: React.FC = () => { | ||
| return ( | ||
| <ScrollablePane styles={{ root: { marginTop: 60 } }}> | ||
| <div css={{ paddingTop: 20, paddingLeft: 27, paddingRight: 20 }}> | ||
| <h3 style={h3Style}>{formatMessage('Get started')}</h3> | ||
| <ul style={ulStyle}> | ||
| <li style={liStyle}> | ||
| <Link href={linkToGetStarted} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Get started with Bot Framework Composer')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToCreateFirstBot} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Create your first bot')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToTutorials} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Composer tutorials')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToEmulator} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Setting up Bot Framework Emulator')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToQNA} target="_blank" onClick={linkClick}> | ||
| {formatMessage('QnA Maker introduction')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToPackages} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Working with packages')} | ||
| </Link> | ||
| </li> | ||
| </ul> | ||
|
|
||
| <h3 style={h3Style}>{formatMessage('Quick references')}</h3> | ||
| <ul style={ulStyle}> | ||
| <li style={liStyle}> | ||
| <Link href={linkToAdaptiveExpressions} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Learn about Adaptive expressions')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToPreBuiltExpressions} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Find pre-built Adaptive expressions')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToLUFileFormat} target="_blank" onClick={linkClick}> | ||
| {formatMessage('LU file format and syntax')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToLGFileFormat} target="_blank" onClick={linkClick}> | ||
| {formatMessage('LG file format and syntax')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToImport} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Importing a publishing profile')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToPVA} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Integrating with Power Virtual Agents')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToMigrate} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Migrating to Composer')} | ||
| </Link> | ||
| </li> | ||
| <li style={liStyle}> | ||
| <Link href={linkToFeatureRequest} target="_blank" onClick={linkClick}> | ||
| {formatMessage('Submit a feature request')} | ||
| </Link> | ||
| </li> | ||
| </ul> | ||
| </div> | ||
| </ScrollablePane> | ||
| ); | ||
| }; |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: I see a lot of specific recognizer flags. We should consider having things like requiredRecognizers: string[], but it would have to be across the whole codebase.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is about requiring the Azure resource related to the recognizer, which right now is a pretty fixed list. HOWEVER, I agree, it would be nice for recognizers to automatically manifest the resources they require and populate this and other places.
one day!