AmericanRedCross / damage-assessment-bot

Chatbot for immediate damage and needs assessments
BSD 3-Clause "New" or "Revised" License
7 stars 3 forks source link

Adminstack adaptivecards #155

Closed blueelvis closed 5 years ago

blueelvis commented 5 years ago

Hey Max!

I have added the dialogs and adaptive cards for confirming the admin stack. Could you please review and once are happy with the PR, we can merge this :)

-Pranav

maxnorth commented 5 years ago

@blueelvis I just landed the frontend localization which has a solution for localizing the admin stack names. Can you apply the same solution to this so that the admin stack selector is localized?

Also, can you update RcdaTypedSession so that it has a strongly typed interface for userData?

blueelvis commented 5 years ago

@maxnorth - In the conflicting file, I had to replace myanmarAdminStackJson.default to myanmarAdminStackJson to get it working and compiling. Please confirm if this was a typo and then merge the branch into Dev. I have pushed another commit with the latest changes.

blueelvis commented 5 years ago

@maxnorth - Done.