Skip to content

Conversation

@lukabratos
Copy link
Contributor

It's not needed anymore since we've released Beams.

It's not needed anymore since we've released Beams.
@lukabratos lukabratos requested a review from leesio December 12, 2018 10:29
@leesio
Copy link
Contributor

leesio commented Dec 14, 2018

@lukabratos do we know whether people are still using this functionality?

@lukabratos
Copy link
Contributor Author

There might be some people who're still using this functionality but it's alright because they've probably already configured everything that's needed. For new customers that want this functionality we recommend Beams instead.

Reasons for this PR:

  • Links are broken. They're redirecting to Beams and not to the Push Notifications BETA docs.
  • We have Beams now which is improved and better service so we would like to reduce confusion by having docs for Push Notifications BETA and Beams. We want to encourage our customers to use new service which is supported better and provides more value for them.
  • We have migration guide with detailed instructions how to migrate from Push Notifications BETA to Beams.
  • If customers will still have questions they can reach us on community Slack or through support.
  • We have already removed this instructions from other libraries: Ruby, PHP, Python, ...

@leesio
Copy link
Contributor

leesio commented Dec 14, 2018

Sorry! I should have clarified. I'm fine with your change. I was just curious/

@lukabratos lukabratos merged commit 574abf3 into master Dec 14, 2018
@lukabratos lukabratos deleted the update-docs branch December 14, 2018 10:53
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.

3 participants