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

F#1237 same data add in metadata when create a datasource #1559

Merged

Conversation

ninezero90hy
Copy link
Contributor

@ninezero90hy ninezero90hy commented Mar 4, 2019

Description

Related Issue : #1237 #1245

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project. it will be added soon
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document. it will be added soon
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Additional Context

(N/A)

ninezero90hy and others added 30 commits February 28, 2019 14:23
 - Change the signature for use as a projection option
@deidera08
Copy link

@ninezero90hy
Description 입력시 Text를 클릭하면 바로 Edit 화면으로 넘어갈 수 있도록 부탁드려요

@brandon-wonjune @ninezero90hy
그리고 Description 입력부분이 가로 100%이다보니 입력/취소 버튼의 동선이 멀어서 왼쪽 정렬되면 좋을듯 합니다.

@ninezero90hy
Copy link
Contributor Author

@ninezero90hy
Description 입력시 Text를 클릭하면 바로 Edit 화면으로 넘어갈 수 있도록 부탁드려요

@deidera08 수정했습니다

@brandon-wonjune
Copy link
Contributor

@ninezero90hy @brandon-wonjune

데이터소스에 설명이 없는 경우 설명이 없습니다문구가 보여야 하지 않을까요?

넵 설명이 없을 경우 "No description" "설명이 없습니다"문구를 placeholder로 부탁드립니다.

수정하였습니다.

@ninezero90hy
Copy link
Contributor Author

ninezero90hy commented Mar 6, 2019

@brandon-wonjune @ninezero90hy
그리고 Description 입력부분이 가로 100%이다보니 입력/취소 버튼의 동선이 멀어서 왼쪽 정렬되면 좋을듯 합니다.

@brandon-wonjune @deidera08 수정했습니다

2019-03-06 11 43 26
2019-03-06 11 43 42

@kyungtaak
Copy link
Contributor

@ninezero90hy 순서관련 로직이 빠져있어 수정하였습니다.

kyungtaak and others added 2 commits March 7, 2019 10:22
…d_in_metadata_when_create_a_datasource

# Conflicts:
#	discovery-frontend/src/assets/images/icon_info3.png
@alchan-lee
Copy link
Contributor

run build
deploy to 1

@deidera08
Copy link

@brandon-wonjune
Datasource > information에 Dictionary 정보가 잘못 노출되고 있습니다.
datasourceinformation

위 화면과 더불어 Dashboard > Datasource 정보 > Column detail에도 동일 적용 부탁드립니다.

@brandon-wonjune
Copy link
Contributor

brandon-wonjune commented Mar 7, 2019

@brandon-wonjune
Datasource > information에 Dictionary 정보가 잘못 노출되고 있습니다.
위 화면과 더불어 Dashboard > Datasource 정보 > Column detail에도 동일 적용 부탁드립니다.

2019-03-07 5 35 35

수정하였습니다.

@deidera08
Copy link

@kyungtaak @ninezero90hy

@kyungtaak

  • 데이터소스 생성시 자동 생성되는 메타데이터의 디스크립션이 데이터소스 이름으로 저장되고 있습니다
    확인 부탁드립니다
  • 현재 메타데이터 디스크립션에 데이터소스 이름이 저장되고 있음
  • 데이터소스 디스크립션이 메타데이터의 디스크립션에 저장되도록 수정 필요

2019-03-07 5 53 44
메타데잍이터의 설명이 데이터소스의 이름값이 노출되고 있는데, 이 부분이 정리되면 이 이슈는 목적한 기능을 달성한 것 같습니다.
확인부탁드립니다.

@ninezero90hy
Copy link
Contributor Author

ninezero90hy commented Mar 7, 2019

@kyungtaak @ninezero90hy

@kyungtaak

  • 데이터소스 생성시 자동 생성되는 메타데이터의 디스크립션이 데이터소스 이름으로 저장되고 있습니다
    확인 부탁드립니다
  • 현재 메타데이터 디스크립션에 데이터소스 이름이 저장되고 있음
  • 데이터소스 디스크립션이 메타데이터의 디스크립션에 저장되도록 수정 필요

2019-03-07 5 53 44
메타데잍이터의 설명이 데이터소스의 이름값이 노출되고 있는데, 이 부분이 정리되면 이 이슈는 목적한 기능을 달성한 것 같습니다.
확인부탁드립니다.

@kyungtaak 댓글 확인 부탁드립니다

@kyungtaak
Copy link
Contributor

@ninezero90hy 수정했습니다.

@kyungtaak kyungtaak merged commit 011d5c2 into master Mar 7, 2019
@alchan-lee alchan-lee deleted the f#1237-same_data_add_in_metadata_when_create_a_datasource branch March 8, 2019 10:08
@joohokim1
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@datasource Component Name : Datasource/Connection Management @metadata Component Name : Metadata testbed1 Review Test Server 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants