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

Any testing mode available? #2

Open
danielsalare opened this issue Oct 5, 2014 · 8 comments
Open

Any testing mode available? #2

danielsalare opened this issue Oct 5, 2014 · 8 comments

Comments

@danielsalare
Copy link

No description provided.

@jimmyjose-dev
Copy link
Owner

Not Yet :( but will have it soon.

@danielsalare
Copy link
Author

Hi, I'm not able to see if it's working. I have change some of the values but yet I haven't got the alert yet from the demo project, please let me know if I'm missing something or there are any additional steps.

@jimmyjose-dev
Copy link
Owner

I wont be able check on this until weekend, but i am adding @Plopix .. in case he has time he might do the needful

@Plopix
Copy link
Contributor

Plopix commented Oct 7, 2014

Hi, I'm not sure that I'll have a lot of time ( I need to finish my app ;) ) but @edse83 first time I have installed this lib the Alert was never triggered due to the first correction I made on this repository. ( view is not in the window hierarchy error on iOS8 )
Did you get the last code ?

@danielsalare
Copy link
Author

Hi,
I just ran the project and I get an error

fatal error: unexpectedly found nil while unwrapping an Optional value
(lldb)
the issue is here:
var firstUse = prefs.objectForKey(kFirstUseDate) as NSDate

@jimmyjose-dev
Copy link
Owner

I just tested with Xcode 6.0.1 and iOS 7 and 8 and it works fine.
I made few changes, please download the latest code, then in RateMyApp.swift change the line
var promptAfterUses = 10 to var promptAfterUses = 1
then run the app and see if it shows an alert.

@danielsalare
Copy link
Author

Awesome! Its now working :D thanks a lot!

@jimmyjose-dev
Copy link
Owner

Kudos !!!

jimmyjose-dev pushed a commit that referenced this issue Jan 29, 2018
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

No branches or pull requests

3 participants