Skip to content
This repository was archived by the owner on Aug 10, 2026. It is now read-only.

EDU-1561: Adds 'push meta channel' info under a new 'error handling' section - #2192

Merged
m-hulbert merged 1 commit into
mainfrom
EDU-1561--Add-push-meta-channel
Aug 27, 2024
Merged

EDU-1561: Adds 'push meta channel' info under a new 'error handling' section#2192
m-hulbert merged 1 commit into
mainfrom
EDU-1561--Add-push-meta-channel

Conversation

@franrob-projects

@franrob-projectsfranrob-projects commented May 22, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

This PR:

  • Adds 'push meta channel' info under a new 'error handling' section
  • Updates and improves /metadata/subscribe [meta]log content

EDU-1561: Add mention of push meta channel

@ably-ci
ably-ci temporarily deployed to ably-docs-edu-1561-add--ty5cpg May 22, 2024 08:42 Inactive
m-hulbert
m-hulbert reviewed Jun 4, 2024
View reviewed changes
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile Outdated
@franrob-projects
franrob-projects temporarily deployed to ably-docs-edu-1561-add--znyksq June 6, 2024 12:52 Inactive
@franrob-projects
franrob-projects temporarily deployed to ably-docs-edu-1561-add--otnckl June 12, 2024 10:25 Inactive
@franrob-projects
franrob-projectsforce-pushed the EDU-1561--Add-push-meta-channel branch from 1e4dfaa to ec92f5dCompareJune 24, 2024 13:15
@franrob-projects
franrob-projectsforce-pushed the EDU-1561--Add-push-meta-channel branch from 8a08585 to 6e3723eCompareJune 24, 2024 13:30
m-hulbert
m-hulbert suggested changes Jul 5, 2024
View reviewed changes

@m-hulbertm-hulbert left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we need to be clearer about the errors being returned here, and make the example relevant to push.

Comment threadcontent/metadata-stats/metadata/subscribe.textile Outdated
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile Outdated
@franrob-projects
franrob-projectsforce-pushed the EDU-1561--Add-push-meta-channel branch from db94aac to 752e3c2CompareJuly 8, 2024 15:55
m-hulbert
m-hulbert suggested changes Jul 17, 2024
View reviewed changes
Comment threadcontent/metadata-stats/metadata/subscribe.textile Outdated
Comment threadcontent/metadata-stats/metadata/subscribe.textile Outdated
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile
Comment threadcontent/metadata-stats/metadata/subscribe.textile Outdated
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile Outdated
Comment threadcontent/push/index.textile Outdated
@franrob-projects
franrob-projectsforce-pushed the EDU-1561--Add-push-meta-channel branch 2 times, most recently from 4c98495 to 574c185CompareJuly 18, 2024 16:53
@franrob-projects
franrob-projectsforce-pushed the EDU-1561--Add-push-meta-channel branch from e97db87 to a4354e8CompareJuly 22, 2024 16:30
@franrob-projects
franrob-projects temporarily deployed to ably-docs-edu-1561-add--no5w0j July 22, 2024 16:32 Inactive
@m-hulbert
m-hulbert merged commit 7354419 into mainAug 27, 2024
@m-hulbert
m-hulbert deleted the EDU-1561--Add-push-meta-channel branch August 27, 2024 13:30
@coderabbitaicoderabbitaiBot mentioned this pull request Nov 13, 2025
Merged
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@m-hulbertm-hulbertm-hulbert approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@franrob-projects@m-hulbert@ably-ci