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

Update sugar for multi-dex #327

Merged
merged 5 commits into from
Oct 27, 2015
Merged

Update sugar for multi-dex #327

merged 5 commits into from
Oct 27, 2015

Conversation

tracytheron
Copy link
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
Contributor Author

I added some missed imports

@coveralls
Copy link

Coverage Status

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

@sibelius
Copy link
Contributor

@tracytheron could you fix the conflicts please?

@tracytheron
Copy link
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
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
Contributor

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

@Vookash
Copy link
Contributor

Vookash commented Oct 27, 2015

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

@sibelius
Copy link
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