Conversation
TShapinsky
left a comment
There was a problem hiding this comment.
This looks really good Hannah. I think we'll probably have some questions in the future about what exactly we want certain strings to be in the UI. But, I think it's more important to get this PR merged soon. What is the status of the model namespace prepending when evaluating a template in the UI? Is that something which was intended for this PR or for a future one?
buildingmotif-app/src/app/template-evaluate/template-evaluate.component.ts
Outdated
Show resolved
Hide resolved
We can add it in this PR- I'm just not sure how to do it... |
I think minimally that's about is, there's still a question about the separator. Maybe just use a |
9837b54 to
549e2ef
Compare
TShapinsky
left a comment
There was a problem hiding this comment.
Besides the mistake I made this looks good to me, @haneslinger
|
This passes locally 😿 |
Moves the evaluate ui to the model detail page.


