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

에디터 컴포넌트 등에서 extravars 항목 중 type이 설정되지 않은 경우에 대한 수정 #1872

Merged
merged 1 commit into from Feb 4, 2022

Conversation

misol
Copy link
Member

@misol misol commented Feb 4, 2022

에디터 컴포넌트 등의 설정 xml 에서 type이 설정되지 않은 경우 text 가 기본 값으로 합니다.
이슈: #1871
@Lastorder-DC 님 감사합니다.

에디터 컴포넌트 등의 설정 xml 에서 type이 설정되지 않은 경우 `text` 가 기본 값.
#1871
@bjrambo
Copy link
Member

bjrambo commented Feb 4, 2022

미솔님 master브랜치에서 수정하셧는데요..?ㅋㅋㅋ
수정diff는 문제 없어보이긴한데.. ;ㅁ; develop브랜치를 따로 빼셔서 작업해두시징 ㅎㅎ

ㅋㅋㅋ 크게 상관없을듯 하네요 diff가 많은게 아니라서 ㅋㅋ

@kijin
Copy link
Member

kijin commented Feb 4, 2022

미솔님의 master 브랜치에서 작업하셨지만 PR은 develop으로 작성하셨으니 상관없을 것 같습니다.
나중에 본인 저장소에서 커밋 순서 정리하는 것은 숙제로... ㅋㅋㅋ

@misol
Copy link
Member Author

misol commented Feb 4, 2022

@bjrambo rhymix:develop 으로 커밋을 넣었는데, misol:master는 저도 실수.. 다시 하는게 나을까요?

@bjrambo
Copy link
Member

bjrambo commented Feb 4, 2022

@misol 상관없을 것 같아요 ㅋㅋㅋ

@misol
Copy link
Member Author

misol commented Feb 4, 2022

@kijin
감사합니다!

@misol
Copy link
Member Author

misol commented Feb 4, 2022

한 단어 넣은거지만, 그래도 framework에 있는 수정이고, 너무 오랜만이라서 며칠 조금 둘게요!

@bjrambo
Copy link
Member

bjrambo commented Feb 4, 2022

_getExtraVars 함수 호출하는 항목들 찾으셔서 혹시 문제있는지 꼼꼼하게 테스트해보셔요 ㅎㅎ

@misol
Copy link
Member Author

misol commented Feb 4, 2022

@bjrambo 넵! 호출하는 곳을 일단 다 찾아보겠습니다.

@misol
Copy link
Member Author

misol commented Feb 4, 2022

@bjrambo 넵! 호출하는 곳을 일단 다 찾아보겠습니다.

이렇게 비장하게 말했는데, *.php 중에 _getExtraVars 는 딱 세 번 나오네요 ㅎㅎ
image

@bjrambo
Copy link
Member

bjrambo commented Feb 4, 2022

@misol 에디터 컴포넌트에서만 쓰네요 ㅎㅎ 문제 없을듯

@misol
Copy link
Member Author

misol commented Feb 4, 2022

@bjrambo 그러게요!

@misol
Copy link
Member Author

misol commented Feb 4, 2022

생각보다 광범위하게 참조하는 함수가 아니고, 사용상 문제가 없는 것으로 보여서 merge 하겠습니다.
의견 주신 @Lastorder-DC 님, @kijin 님, @bjrambo 님 감사합니다.

@misol misol merged commit 21c1763 into rhymix:develop Feb 4, 2022
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

3 participants