Skip to content
This repository was archived by the owner on Jan 14, 2018. It is now read-only.

Add a robospice-retrofit2 module#459

Open
mykolaj wants to merge 1 commit intostephanenicolas:masterfrom
mykolaj:robospice-retrofit2
Open

Add a robospice-retrofit2 module#459
mykolaj wants to merge 1 commit intostephanenicolas:masterfrom
mykolaj:robospice-retrofit2

Conversation

@mykolaj
Copy link
Copy Markdown

@mykolaj mykolaj commented May 7, 2016

Add a robospice-retrofit2 module to be able to use Retrofit-2.0.0 and up with RoboSpice

@sailorseashell
Copy link
Copy Markdown

@mykolaj I looked into why these builds were failing and it looks like it's because you didn't push the changes to your submodule for retrofit2. If you push the submodule changes first, followed by the main repo changes, it should work.

@mykolaj
Copy link
Copy Markdown
Author

mykolaj commented Feb 14, 2017

@AliceHarrison OK, I'm going to need to refresh my memory a little about this issue, and will do everything from my side after that.

@sunnymopada
Copy link
Copy Markdown

sunnymopada commented May 12, 2017

@mykolaj Did you push the code with support of retrofit two?

@mykolaj
Copy link
Copy Markdown
Author

mykolaj commented May 12, 2017

@SanyasiraoM Please follow a link to check out my repo with a demo on how Retrofit2 can be used with Robospice: https://github.com/mykolaj/Robospice-Retrofit2-sample
Although I've created a pull request, I doubt that it would be merged and/or released any time soon.

// ---- BEGIN
In case someone need to use Retrofit2 with RoboSpice take a look at a sample repository.

What you can do is to clone that sample, take '.jar' files from a 'libs' folder from here, and import // them into your own project.
git clone https://github.com/mykolaj/Robospice-Retrofit2-sample.git

That's all for now.
// ---- END

@sunnymopada
Copy link
Copy Markdown

Thank you @mykolaj

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants