-
Notifications
You must be signed in to change notification settings - Fork 763
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
The update_activity python documentation contains errors and no samples for update_activity #1226
The update_activity python documentation contains errors and no samples for update_activity #1226
Comments
Hello @DeanHnter - Thanks for raising your query. |
Hi thanks for looking into it. Here is the link to the original document and the python version is 3.11 |
facing the same issue in update activity. any updates ? |
Looks like the referenced ticket isn't addressing the same issue (superficially) here and considering their documentation isnt even updated to explain that its broken, so not sure if we will see a fix soon im expecting that moving to c# implementation would be easier than waiting on a fix at this point. |
@DeanHnter , we are actively working on this and will let you know the updates. |
Hello @DeanHnter , message = await turn_context.send_activity("") new_activity = MessageFactory.text("The new text for the activity") update_result = await context.update_activity(new_activity) |
Hi thank you for the feedback. Unfortunately the problem hasn't been addressed in any direction.
I cannot recommend anyone actually use this library until the basic functionality actually works and samples that are coherent are produced, it makes no sense that this is the response. |
@DeanHnter , |
@Vaibhav-MSFT any updates on this? facing the same issue |
@sethteo - Apologies for the delay, we will check this and update you soon. |
I am closing this issue. It seems resolved either by upgrading to a newer version of python or by upgrading to the latest (4.16.1) version of bot-builder. Either way for me it now works, the documentation however is still not updated and still contains the incorrect variable names as previously outlined. I no longer care to keep this issue/discussion open as its clear microsoft employees arent capable. I would still recommend migrating to another solution for anyone encountering this issue. |
Tell us about your experience!Hi @DeanHnter! This issue is closed in our system. We would like your feedback on your experience with our support team and Platform. Best regards, |
The current documentation on update_activity for python contains errors and so isnt directly usable without modification, further the referenced example of update_activity is focused on updating a card activity but doesent actually give a example of updating a regular message factory. This would be useful as currently im struggling to find an answer to "error unknown" as a resulting response and it would help to reduce the confusion if the samples were actually functional and demonstrated the API's usage correctly.
An error occurred while updating the activity: (ServiceError) Unknown
context seems like it should be referencing turn_context which would be consistent with other examples however as mentioned this sample does not work for me as update_activity always results in a error: unknown. Further the activity_id seems to reference the variable defined previously activityId, so in short this is a confusing example and I cant get the code to function even if i imply that the error is caused by these mistakes i get an ambiguous error unknown message.
The text was updated successfully, but these errors were encountered: