Skip to content

Rewrite the library to use axios - #2

Merged
ErikBjare merged 21 commits into
masterfrom
dev/axios
Apr 1, 2018
Merged

ErikBjare merged 21 commits into
masterfrom
dev/axios

Conversation

@ErikBjare

@ErikBjare ErikBjare commented Aug 18, 2017 •

Copy link
Copy Markdown
Member

Travis now passes as well, which is nice.

TODO

  • Implement support for the basic APIs
    • Create bucket (needed in aw-watcher-web)
    • List bucket (needed in aw-webui)
    • Delete bucket (needed in aw-webui)
    • List events (needed in aw-webui)
    • Query (needed in aw-webui)
    • Heartbeat (needed in aw-watcher-web)
  • Write tests with mocha

@johan-bjareholt johan-bjareholt self-assigned this Mar 28, 2018
@ErikBjare

Copy link
Copy Markdown
Member Author

@johan-bjareholt Anything left?

I should probably give fixing Travis another go, but I could do that in another PR at a later time.

@ErikBjare ErikBjare changed the title Started rewriting the library with axios Rewrite the library to use axios Apr 1, 2018
@johan-bjareholt

Copy link
Copy Markdown
Member

@ErikBjare As far as I know no. Everything is there except for /log which we are not using anymore and I believe is broken now anyway.
Still no heartbeatqueue for the webextension, but that can be made in a seperate PR.

@ErikBjare

Copy link
Copy Markdown
Member Author

Cool, then I'm merging!

@ErikBjare
ErikBjare merged commit 9aaaf3f into master Apr 1, 2018
@ErikBjare
ErikBjare deleted the dev/axios branch April 1, 2018 14:30
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.

2 participants