Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

[Web Portal] New user management UI. #2726

Merged
merged 7 commits into from
May 16, 2019
Merged

Conversation

mslichao
Copy link
Member

@mslichao mslichao commented May 8, 2019

#2494 in #2078

This PR only change the root page of user management. Will modify other pages when designer gave the new design.

below is the screenshot:

image

@coveralls
Copy link

Coverage Status

Coverage remained the same at 53.075% when pulling 214136a on lchao/newUserManagementUI1 into 0eb286f on master.

@coveralls
Copy link

coveralls commented May 8, 2019

Coverage Status

Coverage decreased (-0.01%) to 53.064% when pulling 4bc196f on lchao/newUserManagementUI1 into 0eb286f on master.

src/webportal/src/app/user/fabric/conn.js Outdated Show resolved Hide resolved
src/webportal/src/app/user/fabric/conn.js Outdated Show resolved Hide resolved
src/webportal/src/app/user/fabric/conn.js Outdated Show resolved Hide resolved
src/webportal/src/app/user/fabric/userView/Ordering.jsx Outdated Show resolved Hide resolved
@mslichao mslichao force-pushed the lchao/newUserManagementUI1 branch from b0acf3b to e44a2ab Compare May 9, 2019 05:37
@mslichao mslichao requested a review from sunqinzheng May 9, 2019 13:57
<Modal
isOpen={showEditInfo.isOpen}
styles={{main: [t.mw8, t.w90]}}>
<div dangerouslySetInnerHTML={{__html: showEditInfo.innerHtml}} />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you refactor the ejs into a react component?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@mslichao mslichao requested a review from sunqinzheng May 11, 2019 08:13
@mslichao mslichao merged commit a774d4d into master May 16, 2019
@mslichao mslichao deleted the lchao/newUserManagementUI1 branch May 16, 2019 03:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants