Skip to content

Fixed Login#6

Closed
chrisjoyce911 wants to merge 14 commits intoRocketChat:masterfrom
chrisjoyce911:master
Closed

Fixed Login#6
chrisjoyce911 wants to merge 14 commits intoRocketChat:masterfrom
chrisjoyce911:master

Conversation

@chrisjoyce911
Copy link

Was
POST https://rocketchat.server/api/v1/login HTTP/1.1 1 1 map[Content-Type:[application/json]] {password=PASSWORD&user=USER}

Now
POST https://rocketchat.server/api/v1/login HTTP/1.1 1 1 map[Content-Type:[application/json]] {{"username":"USER","password":"PASSWORD"}}

Was
 POST https://rocketchat.server/api/v1/login HTTP/1.1 1 1 map[Content-Type:[application/json]] {password=PASSWORD&user=USER}

Now
 POST https://rocketchat.server/api/v1/login HTTP/1.1 1 1 map[Content-Type:[application/json]] {{"username":"USER","password":"PASSWORD"}}
@CLAassistant
Copy link

CLAassistant commented Aug 25, 2018

CLA assistant check
All committers have signed the CLA.

Chris Joyce and others added 13 commits August 25, 2018 21:08
Using multi pagkages makes forked development very hard,
moved things round to have all in single pagkage

* Added Doer - can pass this in
* REST channels has good coverage
* REST information has good coverage
Added missing tests for REST
Included error tests
Added missing tests for REST
Included error tests
Renamed channels tests
Extended the example to include a PostMessage
Most client methods are used by other func during tests
Very simple tests only
Login and Logout
I have left some dead/unsed code that needs cleanup
@geekgonecrazy
Copy link
Contributor

Thanks for your contributions!

It looks like you may have combined changes from #8 into this same PR. So this PR is dependable on #8 being merged

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.

3 participants