Needs more documentation #1

Closed
aaronbrethorst opened this Issue May 26, 2014 · 3 comments

Comments

Projects
None yet
2 participants

I recommend adding something like

#pragma message "Needs filling in!"
#pragma message "Go to http://xyz.com/foo to register an API key"

to your pch file. Also, the stuff in the pch file that needs to be copied to a user's pch file should be moved into a different file

Owner

muhammadbassio commented May 27, 2014

Thanks for the comments, I will modify the needed files when I am back from
my vacation next week.

I will keep you updated once I apply the below recommendations.

On Monday, May 26, 2014, Aaron Brethorst notifications@github.com wrote:

I recommend adding something like

#pragma message "Needs filling in!"
#pragma message "Go to http://xyz.com/foo to register an API key"

to your pch file. Also, the stuff in the pch file that needs to be copied
to a user's pch file should be moved into a different file


Reply to this email directly or view it on GitHubhttps://github.com/muhammadbassio/YouTubeAPI3-Objective-C-wrapper/issues/1
.

👍

Sent from my iPhone

On May 26, 2014, at 7:35 PM, muhammadbassio notifications@github.com wrote:

Thanks for the comments, I will modify the needed files when I am back from
my vacation next week.

I will keep you updated once I apply the below recommendations.

On Monday, May 26, 2014, Aaron Brethorst notifications@github.com wrote:

I recommend adding something like

#pragma message "Needs filling in!"
#pragma message "Go to http://xyz.com/foo to register an API key"

to your pch file. Also, the stuff in the pch file that needs to be copied
to a user's pch file should be moved into a different file


Reply to this email directly or view it on GitHubhttps://github.com/muhammadbassio/YouTubeAPI3-Objective-C-wrapper/issues/1
.


Reply to this email directly or view it on GitHub.

Owner

muhammadbassio commented Jun 3, 2014

Hi Aaron,

I updated the repo with a new version clearing all your concerns.
Thanks to check :).

Regards
​,
Muhammad.​

On Tue, May 27, 2014 at 6:27 AM, Aaron Brethorst notifications@github.com
wrote:

👍

Sent from my iPhone

On May 26, 2014, at 7:35 PM, muhammadbassio notifications@github.com
wrote:

Thanks for the comments, I will modify the needed files when I am back
from
my vacation next week.

I will keep you updated once I apply the below recommendations.

On Monday, May 26, 2014, Aaron Brethorst notifications@github.com
wrote:

I recommend adding something like

#pragma message "Needs filling in!"
#pragma message "Go to http://xyz.com/foo to register an API key"

to your pch file. Also, the stuff in the pch file that needs to be
copied
to a user's pch file should be moved into a different file


Reply to this email directly or view it on GitHub<
https://github.com/muhammadbassio/YouTubeAPI3-Objective-C-wrapper/issues/1>

.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#1 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment