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

facebook share parmameter apply markdown to string #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

uyu423
Copy link
Member

@uyu423 uyu423 commented Jan 25, 2017

git에 있는 것 만으로는 ng serve가 되지 않아서 테스트는 못해봤습니다 ㅜㅜ

closes #11

Copy link
Member

@qkraudghgh qkraudghgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

실제로 돌아가는 코드는 아닙니다.
Typescript 특성상 NPM module을 직접 가져다 쓸땐
Typing해야된다는 것을 저번 슬랙 채팅에서 같이 말씀하신적이 있죠. 이것이 해결되면 Merge하도록 하겠습니다.

기여 감사합니다. 꼭 필요한 기능이긴 해요 ㅠㅠ
아래는 저번에 올려주신 링크입니다.

http://blog.nemo157.com/2014/07/typescript-and-npm-modules.html

http://stackoverflow.com/questions/36309361/using-npm-modules-in-typescript

@@ -8,6 +8,9 @@ import {User} from "../../../model/user.model";
import {AuthService} from "../../../service/auth.service";
import {Observable} from "rxjs";

import remark from "remark";
import md2string from "mdast-util-to-string";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅠㅠ Typing이 안되어있어서 바로 가져다 쓸 수 없는게 좀 충격이네요

@uyu423
Copy link
Member Author

uyu423 commented Jan 26, 2017

흡.. ㅜㅜ 이제 프로젝트 빌드가 되니 시간적 여유가 될 때 타입스크립트 타이핑을 직접 시도해봐야겠군요!

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.

QA: onShareInFacebook() Description Problem
2 participants