Skip to content

Add missing docs snippets and fix marker spacing - #66

Merged
techwritermat merged 2 commits into
masterfrom
docs-snippets-missing-samples
Oct 9, 2026
Merged

techwritermat merged 2 commits into
masterfrom
docs-snippets-missing-samples

Conversation

@techwritermat

Copy link
Copy Markdown
Contributor

Summary

Docs reference snippet ids that do not exist in the samples, which renders a 'Code snippet not found' error.

  • Add get_state_from_group (Sample_Presence)
  • Add set_user_metadata_with_settings and set_channel_metadata_with_settings (Sample_AppContext)
  • Fix //snippet.update_user_metadata_iteratively and update_channel_metadata_iteratively markers in the header, which the parser ignored

Side effect: set_user_metadata_raw in Sample_AppContext.h no longer includes the unrelated UpdateUserMetadataIteratively declarations.

Docs PR: https://github.com/PubNubDevelopers/documentation/pull/1023

Test plan

  • Docs snippet validator finds all ids against this branch
  • Not compiled, no Unreal Engine available locally. Calls were checked against PubnubClient.h and PubnubStructLibrary.h. New samples are added to RunSamples()

🤖 Generated with Claude Code

Add get_state_from_group, set_user_metadata_with_settings and set_channel_metadata_with_settings samples used by the docs. Fix '//snippet.' markers missing a space in Sample_AppContext.h.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@KGronek-Pubnub KGronek-Pubnub 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.

LGTM

GetState for a channel group does not work until the next C-Core release. Drop the sample so docs do not reference code that fails.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@techwritermat
techwritermat merged commit 79245f1 into master Oct 9, 2026
3 checks passed
@techwritermat
techwritermat deleted the docs-snippets-missing-samples branch October 9, 2026 09: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.

2 participants