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

Allow custom Rating UI #170

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Allow custom Rating UI #170

wants to merge 2 commits into from

Conversation

xialin
Copy link

@xialin xialin commented Mar 14, 2014

Added ShowRatingAlert flag. When set No, default rating alert will not show. Instead, you can add a custom page for rating.

@ay8s
Copy link

ay8s commented May 6, 2014

👍 Thanks @xialin! Was looking at doing this myself to use our own alert code.

@aplourde
Copy link

aplourde commented May 7, 2014

👍 was about to let down appirater because of this missing feature ! Thanks man !
I would do the callback with the AppiraterDelegate though instead of a notification.

@coneybeare
Copy link

UAAppReviewManager does this a little bit better

@ay8s
Copy link

ay8s commented May 7, 2014

Oh didn't know about UAAppReviewManager, thanks @coneybeare!

@aplourde
Copy link

aplourde commented May 7, 2014

However it doesn't support using our own view instead of the UIAlertView. At least it's mentioned in the "What's Planned" section.

@coneybeare
Copy link

@aplourde Yes it does. You have callback blocks that can stop presentation of the alertview and you can present your own. The What's Planned section is talking about a custom UIAlertView with the same API so that a simple "alertClass" can be passed upon config.

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.

4 participants