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

Fixes #646 : App crash fixed on Share click #684

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

Conversation

NikunjGarg2000
Copy link

Fixes #646

Changes: [Add here what changes were made in this issue and if possible provide links.]

It was crashing because of NULL pointer exception therefore I added a check if that is NULL and it now works smoothly

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing: [Compress the app-debug.apk file into a <feature>.rar or <feature>.zip file and upload it here]
app-debug.zip

@NikunjGarg2000
Copy link
Author

NikunjGarg2000 commented Feb 13, 2021

@jddeep , @adityastic, @CloudyPadmal please review this one.

@NikunjGarg2000
Copy link
Author

Also #650 is the same issue.

@pratik9315
Copy link

hello sir, I saw this issue is open, can i work on it? If this issue is already assigned, sorry about bothering you, im new to contributing on fossiasia

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.

Share Button on Navigation Drawer
2 participants