관리자 회원목록 페이지에서, 이메일이 이중으로 중복 출력되는 버그 수정

[sejin7940]

관리자 회원목록 페이지에서, 이메일이 이중으로 중복 출력되는 현상 발생
tpl 파일에서, email 을 강제로 제일 앞에 출력하고 있는데
또 $usedIdentifiers 에서 이메일을 한번 더 출력하게 되어있어서 ( 이메일은 필수기재사항이다보니, 무조건 이 변수에 포함되기에)

tpl 의 제일앞의 이메일에는 회원팝업 이 연결되어있어 그건 그대로 두고
대신 $usedIdentifiers 의 기준이 되는 $memberIdentifiers 를 수정해서 이메일을 제외.

[lansi951]

세진님 풀리퀘는 항상 Merge pull request 커밋이 따라 오는데 뭔가 잘못된 건 아닌지...

[bjrambo]

@lansi951 지속적으로 xpress의 develop 브랜치를 받아서 그렇습니다.

보통 사람들이 첫 포크 이후 작업을 한다면 세진님은 기존에 작업햇던내용을 그대로 유지 한 상태에서 develop 브랜치의 브랜치의 내용을 계속 받아오셧기 때문에 생기는 겁니다.

잘못되었다고 보긴 어렵고 신경쓰시지 않아도 될 부분인거죠 :)

[sejin7940]

@lansi951 @qw5414
제가 github 를 잘 몰라서 그런데, Full Request 할때마다 최신 XE develop 브랜치를 다시 merge 해서 해야하는거 아닌가요?
그 사이에 다른분들의 FR 로 XE develop에 새로운 사항이 merge 되어있을 수 있고
그걸 보완 안 하고 예전걸로 하다보면 제가 수정한 부분이 다른 분이 한 FR 을 덮거나 변형시켜버릴 수도 있겠다 싶어서요
어느 방식으로 해야하는게 맞는지 좀 알려주세요 ^^;

전 FR 할때마다 XE develop 을 새로 merge 한 후 그걸 기준으로 제 쪽에서 또 새로 brench 를 만들어서 고치거든요. 결국 FR 수 만큼 제 쪽 brench 수가 생기죠 ㅡㅡ;

[bjrambo]

@sejin7940 당연히 자주자주 폴리퀘 보내시는 상태시라면 당연히 merged 생기는게 당연합니다.

신경쓰시지 않고 원래 하시던 방식대로 폴리퀘 주시면 됩니다.

어쩔수 없어요.
기존의 폴리퀘 보내신 브랜치 에서 계속받으면서 진행하시는거고, 이부분은 XE에서는 큰 신경 안쓰실거에요

중요한건 변화하는 코드가 중요한거지 커밋이 아니거든요

[lansi951]

@sejin7940 그런 거였군요. Git에서 원본 branch가 변경되면 거기서 나온 branch도 변경되는 줄 알았는데 안 되서 난감했습니다 저도..

전 develop에서 branch를 하나 만들어서 거기다 한 뒤에 merge 없이 그냥 풀리퀘 해도 충돌 없이 깔끔히 되더군요.

[bjrambo]

@lansi951 되도록이면 세진님처럼 하시는걸 추천해요.

기존 develop에서 또 수정된내용이 잇을경우 커밋이 꼬일가능성도 배재할수 없기에..ㅎㅎ

[lansi951]

@qw5414 다른 유명한 저장소들을 보니까 안 그런 거 보니 따로 방법이 있는 거 같네요

[bjrambo]

@lansi951 보통 저도 그렇고 새로 저장소를 지웠다가 포크를 해서 받는 편도 있습니다 :)

[lansi951]

@sejin7940 merge 할 때 no commit 옵션이 있네요. 나중에 한번 사용해보시면 좋겠네요.