Skip to content

Update sugar for multi-dex#327

Merged
sibelius merged 5 commits intochennaione:masterfrom
tracytheron:master
Oct 27, 2015
Merged

Update sugar for multi-dex#327
sibelius merged 5 commits intochennaione:masterfrom
tracytheron:master

Conversation

@tracytheron
Copy link
Copy Markdown
Contributor

I use sugar-ORM in my project, when I enable multi-dex, not all the tables will be created. So I did some research on it and this pull request might be a solution. I tested it in my project and it works well.

In order to get all classes from all dexes when we enable multi-dex,we need some updates.
miss some imports
@tracytheron
Copy link
Copy Markdown
Contributor Author

I added some missed imports

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.21%) to 70.69% when pulling d75766c on tracytheron:master into 85d5f3e on satyan:master.

@sibelius
Copy link
Copy Markdown
Contributor

@tracytheron could you fix the conflicts please?

@tracytheron
Copy link
Copy Markdown
Contributor Author

@sibeliusseraphini fixed

sibelius added a commit that referenced this pull request Oct 27, 2015
@sibelius sibelius merged commit ed73852 into chennaione:master Oct 27, 2015
@Vookash
Copy link
Copy Markdown
Contributor

Vookash commented Oct 27, 2015

How can I get this fix? I like SugarORM but this MultiDex issue drives me nuts.

@sibelius
Copy link
Copy Markdown
Contributor

You can get the master version and use it as a module in your project

@Vookash
Copy link
Copy Markdown
Contributor

Vookash commented Oct 27, 2015

Could you give me a .jar? Something is not working for me...

@sibelius
Copy link
Copy Markdown
Contributor

I've uploaded a .jar from master here: https://www.dropbox.com/s/9z85lrud4lgiv6e/sugar-0.7.1.201405082137.jar?dl=0

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.

4 participants