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

V2 refactor #58

Open
wants to merge 61 commits into
base: master
Choose a base branch
from
Open

V2 refactor #58

wants to merge 61 commits into from

Conversation

drewjbartlett
Copy link
Collaborator

No description provided.

drewjbartlett and others added 30 commits February 27, 2018 19:52
Note:
The h1 could have SEO issues with the strong tags around api in rapid.js but looks nice.
I believe this should generate d.ts files automatically but we'll see when we get there.
I've added auth as a key on the Config for now until we can determine if it's optional.
I tossed in some heavy comments about `initializeHttp`

I tried to interpret it to my best.
A few code smells were marked with TODO
This is just the dirty to get it transpiling.
…ile compiling code in .gitignore

(cherry picked from commit 97699ce)
Yi Hou and others added 30 commits April 16, 2018 20:19
See: #41

* Renamed to CustomeRouteOptions
* Put the interface to custome-route.ts, so we can kill
custom-route.d.ts
* Adopt change in all other files
#41
* add AuthConfig interface to config.d.ts
* kill auth-config.d.ts
* adopt the changes
See: #41
* merge RequestType, RequestData into core/request.ts
* remove typings/request.d.ts
See: #41
* relocate to src/core
* adopt changes in depending files
See: #41
* relocate interface to src/core
* remove typings dir
See: #41
* add missing files to commit
See: #41
* add missing files to commit
#41
* This will then compile js files to dist
#41
* Set path to main types rapid.d.ts
add .editorconfig and edit .gitignore
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.

None yet

3 participants