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

Add service for acknowledging subscriptions (#48) #58

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

Conversation

iduuck
Copy link

@iduuck iduuck commented Dec 11, 2020

Tasks for this PR:

  • Implement Subscription Acknowledgements
  • Write tests for subscription acknowledgements

@iduuck iduuck marked this pull request as draft December 11, 2020 17:12
@iduuck iduuck force-pushed the feature/subscription_acknowledgement branch 3 times, most recently from 960e93b to d9172f0 Compare December 11, 2020 17:25
@iduuck iduuck force-pushed the feature/subscription_acknowledgement branch from d9172f0 to 89fa629 Compare December 11, 2020 17:26
@iduuck iduuck marked this pull request as ready for review December 11, 2020 17:45
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 98.741% when pulling 9b18f4a on use-pupy:feature/subscription_acknowledgement into ffc2415 on jnbt:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 98.741% when pulling 9b18f4a on use-pupy:feature/subscription_acknowledgement into ffc2415 on jnbt:master.

@ToTenMilan
Copy link

ToTenMilan commented Feb 8, 2021

Thank you @iduuck and candy_check team for your great work. @jnbt Do you plan to merge this PR anytime soon?

@iduuck
Copy link
Author

iduuck commented Mar 1, 2021

@ToTenMilan I am currently using this PR myself in a production application/API. You can use the PR instead of the current candy_check gem using the following line(s) in your Gemfile:

gem 'candy_check', git: 'https://github.com/use-pupy/candy_check.git',
                   branch: 'feature/subscription_acknowledgement'

Disclaimer: While I am in no will of spoofing anything, it is a general disclaimer, that you should always be sure, to check the software you are installing on your system/software. So be sure to check the branch out and what was edited and only then install it into your software.

@courtsimas
Copy link

@jnbt any chance this is worthy of being pulled in? I would love it in the primary gem instead of using a forked version.

@iduuck
Copy link
Author

iduuck commented Jun 5, 2021

I hope it will be merged at some point. For now using my branched version in a production-ready application.

@courtsimas
Copy link

@iduuck Since the official candy_check gem already does acknowledgments and you can pass in a subscription id, like

result = acknowledger.acknowledge_product_purchase(
  package_name: "my-package-name",
  product_id: "my-subscription-id",
  token: "my-token"
)

(according to the readme),... then what does this PR provide that's different? I'm naive and currently implementing IAP for iOS, and about to dive into google/android land so I'm just curious.

@iduuck
Copy link
Author

iduuck commented Jun 8, 2021

@courtsimas So, actually while it might work out (Spoiler: It didn't for me.) for some time, there could be a place in time, where the subscription will not be acknowledged anymore, because Google might cancel the ability to use the product acknowledgement endpoint for the subscriptions.

Looking at the Google Play API's and the Ruby Library this library is using, you are seeing, that there are actually two different methods for acknowledgment (Product ack. & Subscription ack.).

TLDR: This being said, I think, the endpoint (and the methods) might work for you, however, I would not rely my production code on those.

@courtsimas
Copy link

Fair enough. Makes sense. Like I mentioned, I haven’t done this yet - just now diggin in and planning my route. This is good to know.

@fabn
Copy link

fabn commented Jul 7, 2021

@jnbt this would be a great addition to this gem, could you please take a look at it?

Thanks.

@iduuck
Copy link
Author

iduuck commented Jul 9, 2021

Very sad to still see this open...

@styrken
Copy link

styrken commented Sep 29, 2021

Please merge this in already!

Copy link
Collaborator

@christophweegen christophweegen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @iduuck thanks for your contribution and sorry for the very late feedback.

Could you maybe please add a short section in the README with an example so this functionality is documented somewhere?

Once this is done this should be ready to be merged.

@jnbt what do you think?

@iduuck
Copy link
Author

iduuck commented Dec 13, 2021

Hey @christophweegen, would be awesome to get the opinion of @jnbt first, before putting more effort into the PR.

@jnbt
Copy link
Owner

jnbt commented Dec 13, 2021

@iduuck I agree. Everything looks fine. Just some instructions how to use it, would be really useful.

@alhassanreem
Copy link

@iduuck @jnbt Any update on this merge, it would be really useful to have this on the main gem

@mike927
Copy link

mike927 commented Jan 27, 2023

@jnbt Are there any obstacles to merge this PR? I would love to have to as acknowledging subscriptions is crucial part and it's missing now.

@iduuck
Copy link
Author

iduuck commented Feb 1, 2023

I am not using the library anymore actively, so I am not very eager to write a doc on this one. Would be awesome to get the docs written by someone after digging into the code here.

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.

10 participants