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

SUBMARINE-596. [WEB] Implement environment list through UI #376

Closed
wants to merge 3 commits into from

Conversation

kobe860219
Copy link
Contributor

@kobe860219 kobe860219 commented Aug 13, 2020

What is this PR for?

  • Implement get environment data from database.
  • Show environment list through UI.

What type of PR is it?

[Feature]

Todos

What is the Jira issue?

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

How should this be tested?

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

Screenshots (if appropriate)

截圖 2020-08-16 上午9 45 00

Questions:

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

@kobe860219
Copy link
Contributor Author

@pingsutw @liuxunorg Please help me review this pr. Thanks a lot :))

@pingsutw
Copy link
Member

@kobe860219 I found that the environment ID is empty in the screenshot.

@kobe860219
Copy link
Contributor Author

@pingsutw I have removed the column of environment ID.

Copy link
Member

@pingsutw pingsutw left a comment

Choose a reason for hiding this comment

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

+1, LGTM

Copy link
Member

@jiwq jiwq 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 for the contributions. Thanks @pingsutw help to review. Commit it.

@asfgit asfgit closed this in adcc77b Aug 17, 2020
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.

3 participants