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

Bug fix #5 #9

Merged
merged 5 commits into from
Dec 12, 2016
Merged

Bug fix #5 #9

merged 5 commits into from
Dec 12, 2016

Conversation

ztirom
Copy link

@ztirom ztirom commented Dec 11, 2016

This fix resolves #5 “Android App does not show profile images”

screen shot 2016-12-11 at 16 23 54

This fix resolves robinmanuelthiel#5. “Android App does not show profile images”
@robinmanuelthiel
Copy link
Owner

Hi Moritz, thanks a lot for your work. Please help me to understand, what you did here. As far as I can see, you just

  • Created an impl. of IHttpService in the Android project, that uses the native HTTP implementation
  • Provide this impl. to the ConferenceService
  • Enabled the INTERNET permission in the Android Manifest

What I don't get here: Why does this enable the image download? The DroidHttpService is just responsible for downloading the JSON, not the image itself, right? The image is downloaded by the Xamarin.Forms.Image control inside the ImageCell.

So where do you tell the Android system to always use the native HTTP implementation, even in the ImageCell?

@ztirom
Copy link
Author

ztirom commented Dec 11, 2016

Hi Robin-Manuel,

you are right, there is no issue with downloading the data, all it takes is the default implementation of the handler that is declared by the line in the "Environment" txt file:
XA_HTTP_CLIENT_HANDLER_TYPE=Xamarin.Android.Net.AndroidClientHandler

@robinmanuelthiel
Copy link
Owner

Ah okay, I get it, thanks! I am still a little bit confused, because what I already tried to solve this problem was selecting the AndroidClientHandler as the HttpClient Implementation under Project Properties > Android Options. According to this article from Xamarin, it should de exactly the same as adding this Envorionment.txt file. But the settings does not work and your file does. Weird...

Alright, could you please adjust your Pull request, sothat the changes are only three new files in the Android project of the Conference App with Xamarin.Forms, Connecting to the Cloud and Optimizations and Tweaks module? I want to keeps things as simple as possible!

Then it can merge your PR easily and add the desctiption! Again, thanks a lot!

Moritz added 4 commits December 12, 2016 06:42
Removed Droid Implementation, Added EnvironmentVariables.txt to the
other parts of the workshop.
This reverts commit 9cacb6b.
Added EnvironmentVariables.txt, resolves robinmanuelthiel#5
@ztirom
Copy link
Author

ztirom commented Dec 12, 2016

Good Morning, the Environment files are now the only files on the PR.
Thank you and have a nice day.

@robinmanuelthiel
Copy link
Owner

Great! Thank you very much, especially for the effort, but also for the clearification and time! Great work!

@robinmanuelthiel robinmanuelthiel merged commit f49e7d0 into robinmanuelthiel:master Dec 12, 2016
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.

Android App does not show profile images
2 participants