-
Notifications
You must be signed in to change notification settings - Fork 161
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
Create AxiosSearch.kt #23
Merged
Merged
Commits on Nov 25, 2017
-
/** * An example to show how to leverage axios lib to fetch remote data by [email protected] ([email protected]) * (Btw, Vue.js formally leverage axios for ajax related works, so it should be good if React.js also use it) * * Date: Nov 25, 2017 */
Configuration menu - View commit details
-
Copy full SHA for b530fab - Browse repository at this point
Copy the full SHA b530fabView commit details -
Change to common js way per Hypnosphi advice.
Change from // You need input correct axios.min.js link address in public\static\index.html // I personally download axios.min.js proactive in put it into public\static\js folder // <script src="%PUBLIC_URL%/static/js/axios.min.js"></script> // or you can put <script src="https://unpkg.com/axios/dist/axios.min.js"></script> into index.html to avoid proactive download to local // And we provide a simple fun to wrap axios.js, it is not type safe, suggest JB team to provide a formal wrapping for this useful lib to @jsmodule("axios")
Configuration menu - View commit details
-
Copy full SHA for 2668249 - Browse repository at this point
Copy the full SHA 2668249View commit details
Commits on Nov 26, 2017
-
Refactor onChangeFunction to 2 small fun. Add simple type for the axios. I am not familiar with how to add type for external javascript lib. Please feel free to upgrade/enhance from my code directly. Thanks.
Configuration menu - View commit details
-
Copy full SHA for 876a114 - Browse repository at this point
Copy the full SHA 876a114View commit details -
change js{} to jsObject{} for type safe
change js{} to jsObject{} for type safe
Configuration menu - View commit details
-
Copy full SHA for 6dd7c0a - Browse repository at this point
Copy the full SHA 6dd7c0aView commit details
Commits on Nov 27, 2017
-
Add typing both for axios response and error
Add typing both for axios response and error
Configuration menu - View commit details
-
Copy full SHA for e00be55 - Browse repository at this point
Copy the full SHA e00be55View commit details -
Add more typing for axios config/response/error Now, people should be able leverage typed axios lib to more easily co-work with Kotlin-React to create a real app.
Configuration menu - View commit details
-
Copy full SHA for 5464cc3 - Browse repository at this point
Copy the full SHA 5464cc3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7c5167e - Browse repository at this point
Copy the full SHA 7c5167eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 51c634d - Browse repository at this point
Copy the full SHA 51c634dView commit details
Commits on Nov 28, 2017
-
It finally works for geeric type:) Thanks for @Hypnosphi professional advice, does learn something from him!
Configuration menu - View commit details
-
Copy full SHA for d25d3e8 - Browse repository at this point
Copy the full SHA d25d3e8View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.