search

dennisss / tansa

Dancing Robotics Platform
MIT License
39 stars 16 forks source link

Investigate JS frameworks for GUI #89

Closed zls29 closed 7 years ago

zls29 commented 7 years ago

Open ended issue, details tbd...

tessmichi commented 7 years ago

Assigning to Zach per discussion on Requirements doc

dennisss commented 7 years ago

From my experience, I'd recommend React and strongly advise against using anything Angular

tessmichi commented 7 years ago

That's the same thing my old boss suggested, and we discussed it very briefly last term.

Also I wanted to add on this issue that we discussed at our meeting possibly adding TypeScript "Use typescript in GUI as a tool to verify object casting and parsing arguments"

abramowitzK commented 7 years ago

@zls29 we ended up going with React correct? Can we close this issue?

zls29 commented 7 years ago

Correct, this can be closed once #143 is merged.

abramowitzK commented 7 years ago

@zls29 @dennisss Looks like there are build errors on the CI build that are blocking this from being merged.

dennisss commented 7 years ago

Looks like node_modules weren't included in the CI build. My latest commit should hopefully fix it.

abramowitzK commented 7 years ago

@dennisss Fixed it. Build just passed. Thanks

abramowitzK commented 7 years ago

This has been done.