Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem of displaying and editing lg templates in generated Form Dialog #4955

Closed
8 tasks
yeze322 opened this issue Nov 24, 2020 · 0 comments · Fixed by #4986 or #4989
Closed
8 tasks

Problem of displaying and editing lg templates in generated Form Dialog #4955

yeze322 opened this issue Nov 24, 2020 · 0 comments · Fixed by #4986 or #4989
Assignees
Labels
Area: Form Dialogs The creation of an adaptive dialog from a schema defined by creating a form Type: Bug Something isn't working
Milestone

Comments

@yeze322
Copy link
Contributor

yeze322 commented Nov 24, 2020

Describe the bug

Found several problems related to LG displaying and editing a generated dialog in Composer, especially related to the $kind SendActivity.

1. SendActivity node is empty in Flow Editor

Problem:

image

Causes:

It's a bug of the SendActivity node, when the lg template can't be found, it should fallback to the raw value instead of showing nothing.

Suggested solution:

It can be fixed by a one-line change image

2. Form Editor shows the template name instead of the exact content

Problem:

Causes

By design in Composer.

Suggested solution:

Needs furthur discussion.

  • Should we dereference the lg template?
  • If so, how do we show those templates with parameters filled?
  • Or we just keep the current behavior and let users edit the referenced content in all up view?

3. Neither Form Editor can edit the LG content nor the LG all up view page.

Problem:

  1. Form Editor not editable
    image
  2. LG all up view doesn't show lg templates in imported files.

It leads to these templates not editable in entire Composer.

Suggested solutions:

The problem of Form Editor is caused by the generated dialog doesn't include the $designer.id field.
Composer relies on the $designer.id field to associate *.dialog with *.lg files, it makes sure the lg editor work and Action deletion will always clear related lg template.

To fix that, we have several solutions

  1. Let Form Dialog cli follow the $designer.id convention, generate $designer.id when generating dialog.
  2. Update Composer logic to be compatible with the case where $designer.id is absent. We can assigning a designer id only to the modified part.

Version

Browser

  • Electron distribution
  • Chrome
  • Safari
  • Firefox
  • Edge

OS

  • macOS
  • Windows
  • Ubuntu

To Reproduce

Steps to reproduce the behavior:

  1. Create a Form schema, add a property, click on 'generate dialog'
  2. Go to design page, open the - Help() Trigger
  3. Problem 1: nothing shows in the Action card
  4. Problem 2: editing the LG content in Form Editor won't take effect

Expected behavior

Screenshots

Additional context

@yeze322 yeze322 added Type: Bug Something isn't working Needs-triage A new issue that require triage labels Nov 24, 2020
@yeze322 yeze322 changed the title Problem of displaying and editing lg templates in generated dialog of Form Schema Problem of displaying and editing lg templates in generated Form Dialog Nov 24, 2020
@yeze322 yeze322 added the Area: Form Dialogs The creation of an adaptive dialog from a schema defined by creating a form label Nov 24, 2020
@cwhitten cwhitten added this to the R11 milestone Nov 29, 2020
@cwhitten cwhitten removed the Needs-triage A new issue that require triage label Nov 29, 2020
@GeoffCoxMSFT GeoffCoxMSFT self-assigned this Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Form Dialogs The creation of an adaptive dialog from a schema defined by creating a form Type: Bug Something isn't working
Projects
None yet
3 participants