-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Duplicate UI | Bot Ghosting in Teams but works nicely in the Emulator - BotFramework v4 #6616
Comments
I am facing it and can't solve yet! @BruceHaley @Stevenic |
are you able to share a minimal reproducible zipped bot project that I could test locally from my end? If so, please be sure to remove any secrets/keys beforehand. |
Dear @ramfattah Thank you for your alacrity response. I am sharing a senior software engineer @hossainalazad90 with you. He can also share things via Skype if required. He (@hossainalazad90) is a dedicated senior engineer in this project and help you as much as you need. Plus, https://docs.google.com/document/d/1oTS0q3K8_ttKgMtR4RPIKKkq2Uj2BWZslUNtlGFP8Bs/edit : this doc contains the reproduce steps in image gif. If you need anything more, feel free to comment. Thanks again. |
Dear @ramfattah ,I have shared code in google drive here is link https://drive.google.com/file/d/1WYtR6ycBT133gPKzvUeqNKVUPIdLzr9B/view?usp=sharing |
Hi @ramfattah @stevkan @dmvtech @ceciliaavila I have attached a steps that will help to figure it out easily.
Following steps- |
Hi everyone, we have this issue in our backlog and we will start working on it this week. |
Hi everyone, we have successfully reproduced the reported issue by following the provided steps. Debugging the code, we found out that the line 72 is unnecessary. In the provided code, both buttons and choices are being added, which results in the options being duplicated. It is advised to use either buttons or choices, as having both is redundant. Let us know if this solved the issue or if you need anything else. |
Hi @ramfattah, @tracyboehrer, we think this issue can be closed, it was a configuration problem in the hero cards. |
Hey @ceciliaavila Is it fixed? Please fix it, then close it. It is an intense issue, and it took more than 14 hours to get to the bottom of it. We always assume it is us, not the framework; in this, it was the framework and doing GHOSTing. |
I actual provided this workaround on that day to @azaddunite, however, it doesn't look professional when it displays without the double feeding. I think the framework should be smart enough to have single input not double inputs. Hey @azaddunite, Could you please show the diff, how it looks without the line 72 and with. Case # 1Take one screenshot from emulator when we have line 72 Case # 2Take another screenshot without line 72 in the emulator. cc: @ramfattah , @ceciliaavila , @aukgit, @akarim-ofc-outlook |
Hey LuchoPeres Is it fixed yet? |
The same thing works in the Bot Framework Emulator (v4) but in the real team it creates duplicate UI for choices, for the same code it creates a duplicate and doesn’t create one for the other
Version
Describe the bug
The bug is weird and there are similar bugs fixed by the BOT team before. It creates duplicate UI choices for uncertain cases, we debugged it in many different ways and it works nicely in the emulator but creates duplicate choices UI on the actual team communication.
Full Bug Link: https://docs.google.com/document/d/1oTS0q3K8_ttKgMtR4RPIKKkq2Uj2BWZslUNtlGFP8Bs/edit
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
Additional context
I have been developing Microsoft Team ChatBOT using Bot Framework v4 (Packages are mentioned at the end of the document section).
This bot will be added to the Microsoft Team App as an extension. The end user will communicate through this team extension.
According to business, there are some lifecycles that will be done by both Bot and user interaction. The lifecycle is added in a diagram below-
Tracking Status
Dotnet SDK TODO
Javascript SDK TODO
Python SDK TODO
Java SDK TODO
Samples TODO
Docs TODO
Tools TODO
CC
@aukgit , @akarim-ofc-outlook, @BruceHaley, @hossainalazad90, @Stevenic, @EricDahlvang
The text was updated successfully, but these errors were encountered: