-
Notifications
You must be signed in to change notification settings - Fork 40
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
feat: ajax #44
feat: ajax #44
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 완료입니다!
아래 2가지 테스트 케이스도 추가되야겠네요
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
특별한 이견 없습니다! 재성선임님이랑 선임전임님이 달아주신 리뷰 반영해주시면 마무리 될 것 같습니다.
[1/23] 리뷰 완료합니다. 바벨 플러그인 사용하는 부분에서 확인할 사항이 있어서 차주에 다시 여쭤볼게요~ 고생하셨습니다. |
…m-member-expression-literals
Specs
instance options
default options
ajax.defaults.[option]
ajax.defaults.headers.post['Content-Type'] = 'text/plain';
request method aliases
ajax.get(url, options)
ajax.post(url, options)
ajax.put(url, options)
ajax.delete(url, options)
ajax.patch(url, options)
ajax.options(url, options)
ajax.head(url, options)
success response wrapper
error and complete response wrapper
Promise
If the browser supports Promise,
ajax()
returns a Promise object.TO-DO
ajax()
Notice
ajax/index.mjs
: ES6 moduleajax/index.js
: transpiled by Babel