-
Notifications
You must be signed in to change notification settings - Fork 39
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
Change error message in adaptive card for LoginUI failure #77
Conversation
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@@ -337,16 +337,6 @@ internal string GetLoginUITemplate(string loginUIState) | |||
{ | |||
""type"": ""TextBlock"", | |||
""text"": """ + $"{loader.GetString("LoginUI_LoginFailedPage_text2")}" + @""", | |||
""text"": ""Make sure your inputs are correct."", |
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.
Will these texts be localized
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.
Yes, the texts shown on the card are localized. This text present here in the adaptivecard json is a bug. The localized version of this text is being removed as part of this PR to improve the error message as well. Thanks!
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Description
This PR changes the error message on the failing LoginUI per PM suggestions.
Validation
Force replication in debugger to see failure card
Before:
![image](https://private-user-images.githubusercontent.com/128866445/238095019-09546fbe-63bf-4593-a5f2-486504125ce3.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxODc4ODAsIm5iZiI6MTcyMjE4NzU4MCwicGF0aCI6Ii8xMjg4NjY0NDUvMjM4MDk1MDE5LTA5NTQ2ZmJlLTYzYmYtNDU5My1hNWYyLTQ4NjUwNDEyNWNlMy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzI4JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcyOFQxNzI2MjBaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1iZmVjMDc1ODkxMWY3OTlkNGQyYTYyNTMxOWQwN2VlOTU4NjFiNTZlYjQwNjJlMDI3ZTAwNTAzMjg2ZmUzZWFiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.7Noj4m9M-b4ZYtwSYO25_WX-AplnRJcmBpRkSROoV6g)
After:
![image](https://private-user-images.githubusercontent.com/128866445/238095025-c007c19d-cd27-43db-8310-a68fbbddcace.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIxODc4ODAsIm5iZiI6MTcyMjE4NzU4MCwicGF0aCI6Ii8xMjg4NjY0NDUvMjM4MDk1MDI1LWMwMDdjMTlkLWNkMjctNDNkYi04MzEwLWE2OGZiYmRkY2FjZS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzI4JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcyOFQxNzI2MjBaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1mYjIzZDAxNDZkNTdlMmYwZDM5ZmJmYTdiMjAxZTZmMGJiN2U5Y2I1Y2MzYjA5Mjk4NWI4MmNkOTE2ZjY0MmJmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.nTFN1nsQWVs2hq7_OuwkNUU3rcaw8lQyNfOGB1rrSG0)
References
ADO item# 44597703