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

DynamicFieldType: restore TypeConverter lookup #144

Merged
merged 1 commit into from
Mar 10, 2016

Conversation

talklittle
Copy link
Contributor

This was changed between LoganSquare 1.3.3 and 1.3.4.

Reverting this change fixes the compile-time error.

fixes GH-118

EricKuck added a commit that referenced this pull request Mar 10, 2016
DynamicFieldType: restore TypeConverter lookup
@EricKuck EricKuck merged commit 51c81e1 into bluelinelabs:development Mar 10, 2016
@EricKuck
Copy link
Member

Thanks @talklittle! Will be releasing an update that includes this within a few hours.

@EricKuck
Copy link
Member

I actually ended up reverting this PR and fixing it a different way. Since creating TypeConverters can theoretically be expensive, the new change in 1.3.7 should ensure that they're only ever created once. If you have any issues with 1.3.7 vs. 1.3.6, please let me know.

@talklittle
Copy link
Contributor Author

@EricKuck Thanks! 1.3.7 works great. Tested with ProGuard also.

bluef pushed a commit to bluef/LoganSquare that referenced this pull request Oct 11, 2016
* master:
  able to create fatjar
  Reverted bluelinelabs#144 and fixed the issue another way. Also version bumped.
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.

LoganSquare 1.3.4 compile error, "cannot find symbol" with another class as array field
2 participants