Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
e2e69d1
Excluded luconfig.json from project load (#4648)
GeoffCoxMSFT Nov 3, 2020
eacbbd1
Fix: form dialog view dialog action
sorgh Nov 3, 2020
a7b979a
Fix: Prevent dismissing a property card due to click during text sele…
sorgh Nov 3, 2020
2eddc41
Merge branch 'feature/form-dialogs' of https://github.com/microsoft/B…
sorgh Nov 3, 2020
332f06e
fix: Fixes dismissing a property card due to clicking outside of it d…
hatpick Nov 3, 2020
46a7526
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
sorgh Nov 9, 2020
f168890
chore: sync form-dialogs feature branch (#4734)
hatpick Nov 9, 2020
0740fe1
.form extension for form dialog schema files (#4735)
hatpick Nov 9, 2020
54ffa1f
Merge branch 'feature/form-dialogs' of https://github.com/microsoft/B…
sorgh Nov 16, 2020
0fcd752
Merge branch 'main' of https://github.com/microsoft/BotFramework-Comp…
sorgh Nov 16, 2020
5f20d80
Update library, show feature flag
sorgh Nov 16, 2020
79ba8bb
Set app schema for generated dialogs (#4834)
GeoffCoxMSFT Nov 16, 2020
dff29de
Merge branch 'feature/form-dialogs' of https://github.com/microsoft/B…
sorgh Nov 16, 2020
2a2aeb8
Updated strings
sorgh Nov 17, 2020
642f2ed
undo old fix
sorgh Nov 17, 2020
668a19f
revert yarn.lock changes
sorgh Nov 17, 2020
0e9daee
revert generate library update
sorgh Nov 17, 2020
36b3db7
Merge branch 'main' into feature/form-dialogs
hatpick Nov 17, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,10 @@ const EmptyView = styled(Stack)({
opacity: 0.5,
});

type Props = RouteComponentProps<{ projectId: string; schemaId: string }>;
type Props = RouteComponentProps<{ projectId: string; skillId: string; schemaId: string }>;

const FormDialogPage: React.FC<Props> = React.memo((props: Props) => {
const { projectId = '', schemaId = '' } = props;
const { projectId = '', skillId = '', schemaId = '' } = props;
const formDialogSchemaIds = useRecoilValue(formDialogSchemaIdsState(projectId));
const formDialogLibraryTemplates = useRecoilValue(formDialogLibraryTemplatesState);
const formDialogGenerationProgressing = useRecoilValue(formDialogGenerationProgressingState);
Expand Down Expand Up @@ -90,7 +90,7 @@ const FormDialogPage: React.FC<Props> = React.memo((props: Props) => {
const viewDialog = React.useCallback(
(schemaId: string) => {
if (schemaId) {
navigateToGeneratedDialog({ projectId, schemaId });
navigateToGeneratedDialog({ projectId, skillId, schemaId });
}
},
[navigateToGeneratedDialog, projectId]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

import { JsonEditor } from '@bfc/code-editor';
import { FormDialogSchemaEditor } from '@bfc/form-dialogs';
import { FileExtensions } from '@bfc/shared';
import styled from '@emotion/styled';
import { NeutralColors } from '@uifabric/fluent-theme';
import formatMessage from 'format-message';
Expand Down Expand Up @@ -113,7 +114,7 @@ export const VisualFormDialogSchemaEditor = React.memo((props: Props) => {
editorId={`${projectId}:${schema.id}`}
isGenerating={generationInProgress}
schema={schema}
schemaExtension=".form-dialog"
schemaExtension={FileExtensions.FormDialogSchema}
templates={templates}
onGenerateDialog={onGenerate}
onSchemaUpdated={onSchemaUpdated}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -110,8 +110,9 @@ export const formDialogsDispatcher = () => {
}
);

const navigateToGeneratedDialog = ({ projectId, schemaId }) => {
navigate(`/bot/${projectId}/dialogs/${schemaId}`);
const navigateToGeneratedDialog = ({ projectId, skillId, schemaId }) => {
skillId = skillId || projectId;
navigate(`/bot/${projectId}/skill/${skillId}/dialogs/${schemaId}`);
};

const navigateToFormDialogSchema = ({ projectId, schemaId }) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ class FilePersistence {

private getFormDialogSchemaFileChanges(current: FormDialogSchema[], previous: FormDialogSchema[]) {
const changeItems = this.getDifferenceItems(current, previous);
const changes = this.getFileChanges(FileExtensions.FormDialog, changeItems);
const changes = this.getFileChanges(FileExtensions.FormDialogSchema, changeItems);
return changes;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export enum ChangeType {

export enum FileExtensions {
Dialog = '.dialog',
FormDialog = '.form-dialog',
FormDialogSchema = '.form',
DialogSchema = '.dialog.schema',
Manifest = '.json',
Lu = '.lu',
Expand Down
2 changes: 1 addition & 1 deletion Composer/packages/client/src/utils/pageLinks.ts
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ export const topLinks = (
{
to: `/bot/${projectId}/forms`,
iconName: 'Table',
labelName: formatMessage('Forms'),
labelName: formatMessage('Forms (preview)'),
exact: false,
disabled: !botLoaded,
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,11 @@ export const FormDialogSchemaDetails = () => {
const lifetime = new Lifetime();

const clickOutsideLists = (e: MouseEvent) => {
const selection = window.getSelection();

// If click outside is the continuation of text select within the card, don't dismiss the card.
if (selection && selection.toString()) return;

const { x, y } = e;
const elms = Array.prototype.slice.call(
containerRef.current.querySelectorAll(`.${jsPropertyListClassName}`) || []
Expand Down
2 changes: 1 addition & 1 deletion Composer/packages/form-dialogs/src/demo/DemoApp.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -258,7 +258,7 @@ const InternalDemoApp = () => {
allowUndo
editorId={selectedItemId}
schema={selectedItem}
schemaExtension=".form-dialog"
schemaExtension=".form"
templates={templates}
// eslint-disable-next-line no-console
onGenerateDialog={() => {}}
Expand Down
4 changes: 2 additions & 2 deletions Composer/packages/lib/indexers/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class Indexer {
{
[FileExtensions.lg]: [],
[FileExtensions.Lu]: [],
[FileExtensions.FormDialog]: [],
[FileExtensions.FormDialogSchema]: [],
[FileExtensions.QnA]: [],
[FileExtensions.Dialog]: [],
[FileExtensions.DialogSchema]: [],
Expand Down Expand Up @@ -95,7 +95,7 @@ class Indexer {
skills: skillIndexer.index(skillContent, settings.skill),
botProjectSpaceFiles: botProjectSpaceIndexer.index(result[FileExtensions.BotProjectSpace]),
jsonSchemaFiles: jsonSchemaFileIndexer.index(result[FileExtensions.Json]),
formDialogSchemas: formDialogSchemaIndexer.index(result[FileExtensions.FormDialog]),
formDialogSchemas: formDialogSchemaIndexer.index(result[FileExtensions.FormDialogSchema]),
recognizers: recognizerIndexer.index(recognizers),
crossTrainConfig: crossTrainConfigIndexer.index(crossTrainConfigs),
};
Expand Down
2 changes: 1 addition & 1 deletion Composer/packages/lib/indexers/src/utils/fileExtensions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

export enum FileExtensions {
Dialog = '.dialog',
FormDialog = '.form-dialog',
FormDialogSchema = '.form',
DialogSchema = '.schema',
Lu = '.lu',
QnA = '.qna',
Expand Down
2 changes: 1 addition & 1 deletion Composer/packages/lib/shared/src/featureFlagUtils/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export const getDefaultFeatureFlags = (): FeatureFlagMap => ({
FORM_DIALOG: {
displayName: formatMessage('Show Form Dialog'),
description: formatMessage('Show form dialog editor in the canvas'),
isHidden: true,
isHidden: false,
enabled: false,
},
REMOTE_TEMPLATE_CREATION_EXPERIENCE: {
Expand Down
Loading