Skip to content

일부 레이아웃에서 모듈설정의 다국어 레이어가 표시되지 않는 문제점 & 다국어 문서에 존재하는 버그 개선#2212

Merged
2 commits merged into
xpressengine:developfrom
howtoxe:develop
Feb 13, 2018
Merged

Conversation

@howtoxe

@howtoxe howtoxe commented Jan 25, 2018

Copy link
Copy Markdown
Contributor

레이아웃에 id가 content인 요소가 없으면 반투명 배경레이어만 나타나고 다국어 레이어가 로드되지 않는 문제를 수정

@ghost ghost self-assigned this Jan 31, 2018
@ghost ghost added this to the 1.9-2018-12W milestone Jan 31, 2018
@howtoxe howtoxe changed the title 일부 레이아웃에서 모듈설정의 다국어 레이어가 표시되지 않는 문제점 수정 일부 레이아웃에서 모듈설정의 다국어 레이어가 표시되지 않는 문제점 수정 외 Feb 7, 2018
@howtoxe howtoxe changed the title 일부 레이아웃에서 모듈설정의 다국어 레이어가 표시되지 않는 문제점 수정 외 일부 레이아웃에서 모듈설정의 다국어 레이어가 표시되지 않는 문제점 & 다국어 문서에 존재하는 버그 개선 Feb 7, 2018
@howtoxe

howtoxe commented Feb 7, 2018

Copy link
Copy Markdown
Contributor Author

@bnu 머지가 안된 상황에서 새로운 커밋을 다른 pr로 분리하는 방법을 모르겠네요;; 제가 못 찾는건지 원래 깃이란게 이런 시스템인지는 잘 모르겠지만, 어차피 같은 다국어 관련이고 develop 브랜치이니 묶어낼 수 있을 것으로 판단했습니다.

다국어 글/제목 내용 검색이 안되는 부분은 수정해야할 곳들이 좀 많으니 이 pr이 머지되는대로 넣겠습니다.

@Lastorder-DC

Lastorder-DC commented Feb 7, 2018

Copy link
Copy Markdown

@howtoxe PR 자체가 한 브랜치를 다른 브랜치에 병합하는 것을 요청하는 것이기에 PR을 보내신 후 같은 브랜치에 커밋하시면 PR에 포함되게 됩니다. 이것은 의도된 동작으로 서로 다른 PR로 분리하시려면 브랜치를 따로 만드셔야 합니다.

develop 브랜치에서 PR하는 것은 따라서 권장하지 않습니다. develop 브랜치에서 새로 브랜치 생성후 작업하시는것이 좋습니다.(보통 기능추가의 경우 feature/(추가할기능), 버그수정의 경우 bugfix/(버그수정사항)으로 만드는 것이 보통이나 어느 이름이건 상관없습니다.

덧붙여서 커밋 제목은 Update (파일명) 보다는 fix #(이슈번호) (수정사항 요약) 으로 입력하시는 것이 좋습니다.

@howtoxe

howtoxe commented Feb 7, 2018

Copy link
Copy Markdown
Contributor Author

@Lastorder-DC 알려주셔서 감사합니다. XE에나 조금씩 공헌하고 있고 개인용으로 쓴게 다여서 잘 몰랐네요;;

@bnu pr을 다시 나눠서 넣는게 좋을까요?

@ghost ghost merged commit d6b204d into xpressengine:develop Feb 13, 2018
@ghost

ghost commented Feb 13, 2018

Copy link
Copy Markdown

@howtoxe 네. 다음에는 @Lastorder-DC 님이 제시하신 것 처럼 브랜치를 나눠서 PR하시면 됩니다.
이 PR은 두 커밋 모두 반영할 수 있는 사항이라 바로 머지했습니다.
감사합니다 :)

@howtoxe

howtoxe commented Feb 21, 2018

Copy link
Copy Markdown
Contributor Author

@bnu 예, 감사합니다^^

@howtoxe

howtoxe commented Feb 22, 2018

Copy link
Copy Markdown
Contributor Author

@bnu 다국어 게시글의 내용, 제목 검색이 안되는 부분의 경우 pr을 하지 않고 개인적으로 해결하기로 했습니다.

어디까지나 제 생각이긴 합니다만 극히 특수한 케이스를 위해서 일반적인 상황에서의 퍼포먼스를 저하시키는건 맞지 않는 것 같아서요.

This pull request was closed.
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.

2 participants