Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SUBMARINE-676. Add NG-ZORRO UI component library to notebook page #454

Closed
wants to merge 1 commit into from

Conversation

kobe860219
Copy link
Contributor

What is this PR for?

We use NG-ZORRO as the UI Component library in Workbench Web. But the form in notebook page of WEB has not been used yet. We should make the WEB interface all use uniformly.

What type of PR is it?

[Refactoring]

Todos

What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-676

How should this be tested?

https://travis-ci.org/github/kobe860219/submarine/builds/744476551

Screenshots (if appropriate)

x3ie8-uk0nv

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@lowc1012
Copy link
Contributor

@kobe860219
Thanks for your contribution! I tested the functionality of this page. It works as designed.
@kevin85421
Could you review or glance over this PR ? Thanks

@lowc1012
Copy link
Contributor

@kobe860219 Please help to resolve the conflicts. Thanks!

@kobe860219
Copy link
Contributor Author

@lowc1012 I have resolved conflicts :)

Copy link
Contributor

@lowc1012 lowc1012 left a comment

Choose a reason for hiding this comment

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

+1 thanks @kobe860219

@lowc1012
Copy link
Contributor

Will merge this pr if no more comments.

@asfgit asfgit closed this in c360271 Nov 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants