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

Fix BundleResources missing from NuGet package #79

Merged
merged 3 commits into from
Apr 8, 2020

Conversation

Cheesebaron
Copy link
Collaborator

@Cheesebaron Cheesebaron commented Apr 8, 2020

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Bug fix

⤵️ What is the current behavior?

Show methods showing built in error or success images will throw NullReferenceException, because images are missing in NuGet package.

🆕 What is the new behavior (if this is a feature change)?

Added .targets file and added images in NuGet package. These are added to the App at build time.

💥 Does this PR introduce a breaking change?

🐛 Recommendations for testing

Install NuGet in your app and try calling ShowSuccessWithStatus or one of the other methods showing a success or error image.

📝 Links to relevant issues/docs

Fixes #77
Fixes #64

🤔 Checklist before submitting

  • All projects build
  • Follows style guide lines
  • Relevant documentation was updated
  • Rebased onto current develop

@Cheesebaron Cheesebaron added the bug label Apr 8, 2020
@Cheesebaron Cheesebaron merged commit 4fdd2f6 into master Apr 8, 2020
@Cheesebaron Cheesebaron deleted the bugfix/missing-assets branch April 8, 2020 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v 1.3.1 Crash - Xamarin - Targeting 13.2 IOS Icon not showing
1 participant