[GitHub] [rocketmq-dashboard] vongosling edited a comment on pull request #12: [ISSUE #10] add react module for dashboard

[GitBox <gi...@apache.org>]

vongosling edited a comment on pull request #12:
URL: https://github.com/apache/rocketmq-dashboard/pull/12#issuecomment-905089274


   > > Great, we should care for Facebook BSD+Patents license in X category, which is the React prior license. Do we have any optional for here. I still have some concerns about this happening over and over again.
   > 
   > If we want to use React, maybe we can use the current version of React because it uses the MIT license. Once the license is changed we should not upgrade the React version.
   > 
   > React Vue Angular are the most popular front end frameworks, most front end users choose React, I prefer to use it, we can discuss which front end framework to use in this [issue](https://github.com/apache/rocketmq-dashboard/issues/11).
   > 
   > This PR is a solution for building front end resources into spring jar(choose to use node/npm and maven plugin), React is not a must.
   
   Up, If there's nothing wrong with the license, I'm in favor of the react, which is an active and creative community project. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org