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

#34 fixed load error message issue #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shanemckenzie
Copy link

Switched the failed to load error to an alert, prevents the underlying ui from being interacted with while the alert is present and makes the error more readable

@ZiggyKraus
Copy link

Hello @shanemckenzie ,

Krzysztof here, from LiveChat's Technical Support. First of all, our apologies for reaching back to you with such a delay. I have spoken with a person who's handling the Android repository, and we will do our best to review the pull request as soon as possible. Currently, there are few things that we need to take care of in terms of our mobile apps as well, and as a lot of customers uses this repo, we will need to review your PR properly and accept it if possible.

In the meantime, we have got a suggestion: until we will review the PR, maybe you could fork the repo, implement your changes and then publish it (https://proandroiddev.com/android-libraries-on-github-packages-21f135188d58)? You could also implement the changes locally and use them in your project (https://medium.com/stantmob/how-to-use-android-libraries-locally-and-leaving-your-development-more-fast-47ef921c6dd9).

Once again, our apologies for the delay. We will do our best to take a look at it as soon as possible. In the meantime, please don't hesitate to ask in case of any additional questions or concerns.

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.

3 participants