-
Notifications
You must be signed in to change notification settings - Fork 372
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
fix: notification error navigate #2561
Conversation
@zhixzhan why in the scenario of an inline LG error on a template like |
@cwhitten By design it should navigate to inline, current not working come from 2 reason.
for 2 in conclusion, currently, not every inline error can be navigate back to inline editor because of our lg/lu Parser ability. |
I just tried it out and didn't see any different in behavior with a LG error. It still navigated to the all-up page |
can you paste the error lg file or bot, let me check again. if a LG error ( like just updated screenshot ) it did navigate to form editor |
@cwhitten updated, found another reason is lgNamePattern not updated from 'bdfactivity-233534' to 'bdfactivity-23orinfe' |
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.
nice!
* notification navigate fix with locale * navigate with bot id * update id calculate * fix lgNamePattern
Description
Task Item
closes #2533
closes #2546
Screenshots