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

Fix KR Url #4

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Fix KR Url #4

wants to merge 2 commits into from

Conversation

kyechan99
Copy link

@kyechan99 kyechan99 commented Jun 29, 2020

한글 주소 정규식 표현 (#90f5e0d)

Header.js에서 영문이랑 숫자만 허용되도록 거르더군요.
때문에 우측 네비게이션도 작동을 안하고 본문에서도 주소 저장 혹은 바로가기가 안먹히구요.
한글도 허용되도록 정규식 고쳐주고 일부 영문으로된 주소들도 한글로 변경했습니다.

resolver 함수 info 인수 추가 (#860903e)

실행페이지에서 resolver 함수는 네 개의 인수를 받습니다. 라고 하였는데 코드에서는 세 개의 인수만 적혀있더라구요.
원문에서는 info 인수를 모두 적어줬길래 추가해봤습니다.

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.

1 participant