8000
Skip to content

Fix invalid username#67

Merged
fimuxd merged 3 commits intofimuxd:developfrom
kangddong:fix_invalid_username
Apr 11, 2023
Merged

Fix invalid username#67
fimuxd merged 3 commits intofimuxd:developfrom
kangddong:fix_invalid_username

Conversation

@kangddong
Copy link
Copy Markdown
Contributor

invalid username 이슈

원인: 기존 data-count 파싱하던 html 어트리뷰트 키가 바뀜으로서 guard 문에서 return 되어서
수정: html을 파싱하고, [commit Count] + contributions on + [Date] 구조에서 [commit Count]를 count 로 전달

@fimuxd
Copy link
Copy Markdown
Owner
fimuxd commented Apr 11, 2023

부족하다는 말로 부족한 저에게 빛을 주셨었군요. 이마저도 늦게봐서 죄송합니다.
소중한 PR은 합쳐서 반영하겠습니다만 AppStore에 업데이트는 당장 힘들 것 같습니다.
이유는 사실 다른 곳에 있거든요. 😭
그래서 TestFlight 로 외부테스터 초대를 드리려고 합니다.

아무튼..! 수정해주셔서 정말 감사합니다.

@fimuxd fimuxd merged commit 0fd146c into fimuxd:develop Apr 11, 2023
@kangddong
Copy link
Copy Markdown
Contributor Author

이메일 보냈어요 !

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

0