Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Fix PAYLOAD.md regarding "drawable" directory name #1711

Merged
merged 1 commit into from
May 8, 2017

Conversation

gianpaj
Copy link
Contributor

@gianpaj gianpaj commented May 8, 2017

No description provided.

@gianpaj gianpaj mentioned this pull request May 8, 2017
@macdonst macdonst merged commit a27a62d into phonegap:master May 8, 2017
@macdonst
Copy link
Member

macdonst commented May 8, 2017

@gianpaj thanks for the docs clarification.

@gianpaj gianpaj deleted the patch-1 branch May 8, 2017 22:44
@@ -233,7 +233,8 @@ var push = PushNotification.init({
});
```

Where *icon* is the name of an image in the Android *drawables* folder. Writing a hook to describe how to copy an image to the Android *drawables* folder is out of scope for this README but there is an [excellent tutorial](http://devgirl.org/2013/11/12/three-hooks-your-cordovaphonegap-project-needs/) that you can copy.
Where *icon* is the name of an `.png` image file in the Android `res/drawable` folder. For example: `platforms/android/res/drawable/phonegap.png`
Writing a hook to describe how to copy an image to the Android `res/drawable` folder is out of scope for this README but there is an [excellent tutorial](http://devgirl.org/2013/11/12/three-hooks-your-cordovaphonegap-project-needs/) that you dcan copydd
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tail has a couple of typos dcan copydd. I fixed it in 5a4a04b

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks. shouldn't have done the commit / PR so close to bed time 😄

@lock
Copy link

lock bot commented Jun 3, 2018

This thread has been automatically locked.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants