-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
feat: Add appstore review-list
#610
Conversation
* Sort reviews by date descending * Print using fmt.Printf not log.Info, so it is readable in the terminal
and print (responded) if we responded.
* Add `--after` to limit reviews by date * Use sort=-createdDate so we don't have to sort * Use include=response and print body of review if we haven't responded
* Add `--since` to limit reviews by duration * Exit 2 if no reviews to aid scripting
are there going to be other sub-cmds other than |
No I don't think so. I just started using that cmd "in anger" yesterday, and I'm using it to email me a list of all reviews in the last 24h.
But it's not helpful to list apps by app ID so I think I want another command
I see there is already an unused function |
I have no apps w/ reviews. Does this work for you? |
Sure does! I use it in a cron job to mail me any reviews in the last 25 hours like so:
|
* Change copyright statement * Eliminate useless subcommand. Instead of `as review ls` make it `as review-list` (inspired by `device-list`) * Remove commented-out code
All review comments have been addressed @blacktop and it's ready for another review. Importantly, I removed the unnecessary sub-cmd |
LGTM |
Fixes #609
as review-list --id=appid
to list reviews. For now, it uses a simpletext output that is readable in the terminal.