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

RSH1c1 New push->admin->channelSubscriptions->list_ #76

Merged
merged 1 commit into from
Jun 3, 2019
Merged

Conversation

jdavid
Copy link
Contributor

@jdavid jdavid commented May 16, 2019

RSH1c1 New push->admin->channelSubscriptions->list_

list is a reserved word

@jdavid
Copy link
Contributor Author

jdavid commented May 31, 2019

ping

Copy link
Member

@mattheworiordan mattheworiordan left a comment

Choose a reason for hiding this comment

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

LGTM, although if you vary from the spec, and thus violate the docs, you need to make sure the docs repo reflects that change please.

*
* @param array $params the parameters used to filter the list
*/
public function list_ (array $params = []) {
Copy link
Member

Choose a reason for hiding this comment

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

@jdavid if you are using a method that is different to the spec, then you need to update the docs in https://github.com/ably/docs. Can you do a PR for that to update general/push/admin?

@tomczoink see below. Do we need to create an issue for this?

screenshot_2019-05-31_03-27-07_pm

Copy link
Member

Choose a reason for hiding this comment

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

In fact @tomczoink, the problem is worse. Where has Push gone in the documentation nav?? See below for the docs repo nav.

screenshot_2019-05-31_03-29-25_pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants