Remove reviews for in-app sales

[costmo]

Reviews for in-app sales are copies of reviews for the app in which they appear, so having them download and appear in the list of reviews is a waste of time and resources.

I am happy to make the change that excludes in-app purchases from the code that grabs reviews online if someone would provide the information I need to get those changes into the main code branch or get those changes to the developer so that he can commit them. I don't want to have local changes to the code that don't carry over to new revisions.

[evands]

Create a Fork within Github, clone that repository, make your changes, and commit. Then, send us a pull request. :)

[costmo]

I'll do so as soon as I get a chance.

Thanks!

[Paulsmitz]

I have done this and have it working but no one is responding to my pull requests

[evands]

I can't figure out how to handle a pull request in git and haven't had time to try again. Do you know of a straightforward guide? Github's help page's instructions don't work for me.

[Paulsmitz]

Nope never had chance did I send the pull request properly?

[Paulsmitz]

http://www.viget.com/extend/i-have-a-pull-request-on-github-now-what/

Just found this... Looks like it's quite simple...

[evands]

Figured it out. That guide is way overcomplicated for a patch that applies cleanly; you can just go to your Fork Queue, click the checkboxes next to the commits, and click Apply. Done! I've accepted your changes.

[Paulsmitz]

This needs to be reopened... changes have overwritten my changes...

[evands]

Sorry about that, Paul. I did a huge merge with another tree and must have clobbered some part of it accidentally despite attempting to make it clean.

Would it be possible for you to merge with the main repository and send a pull request after you've reapplied whatever change was clobbered?

[Paulsmitz]

I am trying to figure out what has changed... it seems that the whole reports.m has changed... will look into it! :)