-
Notifications
You must be signed in to change notification settings - Fork 58
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
show heart when written on Steemcoinpan #32
show heart when written on Steemcoinpan #32
Conversation
show heart when post is written on nitrous
smaller heart
블록체인스튜디오님 정말 빠르십니다. 저도 같은 기능을 구현했는데, 지금 발견했네요. ㅠㅠ |
add '/' for more secure check
@anpigon님 죄송해하실 필요없습니다ㅠㅠ 나름 기다려도 아무도 안하길래 한건데^^ 딱 겹쳤군요. 보니까 아이콘도 직접 만드신것 같네요. 전 찾아서 쓰려다 svg가 없는 것 같고 또 하트로 해달라고 되어있길래 범용성도 고려해서 하트로 그냥 만들었습니다^^ |
하트가 넘 이쁩니다. 목록뿐만 아니라 상세까지 구현하셨네용. |
감사합니다^^ 저도 문자열 시작부터 체크하는 함수 이름이 생각이 안나서 일단 search로 했다가 안피곤님 코드 보고 startsWith로 바꿨습니다. js는 제가 잘 안써서 기본적인 함수 이름이 잘 생각이 안나네요ㅎㅎ 참 코드를 보니 branch가 나눠져있긴 하지만 문자열 하드코딩은 되도록 피하시는게 좋을 것 같습니다. 그래서 저는 config에서 읽어서 해당 상수로 비교를 했습니다. 라고 말하고 보니 저도 하트를 하트로 그냥 확 적어넣었군요ㅎㅎ |
블록체인스튜디오님 코드를 보고나서 문자열 하드코딩한 것을 후회하고 있습니다. |
덕분에 저도 코드를 수정했습니다. 감사합니다. |
User image proxy for opengraph and twitter images Closes #32 See merge request hive/condenser!44
Show heart when a post is written on nitrous
request from @passionbull
https://steemit.com/sct/@jacobyu/6mg54u