-
Notifications
You must be signed in to change notification settings - Fork 377
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
Previous worker not sending notifications for new Apns Apps #3
Comments
Comment by ileitch from Friday Jul 12, 2013 at 21:59 GMT Yep, this is by design. See https://github.com/ileitch/rapns/wiki/Hot-App-Updates I am tempted to change this behavior though and have the new apps auto-loaded. |
Comment by saulm from Friday Jul 12, 2013 at 22:08 GMT oh, Ok, Thanks for the response |
Comment by saulm from Friday Jul 12, 2013 at 22:26 GMT If someone stumbles accross this, Heroku doesn't support signals |
Comment by ileitch from Monday Jul 15, 2013 at 21:36 GMT Reopening this as a reminder to myself to work on improving this case. |
Comment by y2kRules from Tuesday Oct 08, 2013 at 03:58 GMT So is there anyway to remedy this "issue" on Heroku given the lack of support for signals? Thanks again for this great gem! |
Comment by saulm from Tuesday Nov 19, 2013 at 19:07 GMT @y2kRules Add the heroku-api gem And issue a heroku.post_ps_restart(ENV["HEROKU_APP_NAME"], 'ps' => 'rapns.1') whenever you want to restart the worker |
In 2.0.0 (unreleased) Rpush will automatically load new apps for you. |
Issue by saulm from Friday Jul 12, 2013 at 19:40 GMT
Originally opened as ileitch/rapns#141
Hi @ileitch, first of all thanks for the great work on the rapns gem,
I'm having a small issue and wanted to ask you about it:
When my rapns worker is running and I create a new Apns App, notifications are not send for this new app unless I restart the worker. Is it by design? have you run into this problem before? how have you addressed it?
I'm leaving you a small log sample from my app running on heroku. You can see there that I have to restart the worker so the notifications for the new app are sent.
Thanks in advance,
SM
The text was updated successfully, but these errors were encountered: