Skip to content

Add setPublishableKey#519

Open
ShiCheng-Lu wants to merge 1 commit intomasterfrom
shicheng/set-publishable-key
Open

Add setPublishableKey#519
ShiCheng-Lu wants to merge 1 commit intomasterfrom
shicheng/set-publishable-key

Conversation

@ShiCheng-Lu
Copy link
Contributor

for ephemeral user keys

tested with example app, can set publishable key to different value (UNAUTHORIZED) then back (successful track)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a setPublishableKey method to allow updating the publishable key without re-initializing the SDK, primarily intended for use with ephemeral API keys. The method provides a way to dynamically change the API key at runtime.

  • Adds a new public method setPublishableKey to the Radar SDK interface
  • Includes documentation explaining its purpose for ephemeral API keys

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
RadarSDK/Include/Radar.h Declares the new setPublishableKey method with proper parameter annotations and documentation
RadarSDK/Radar.m Implements the new method by delegating to RadarSettings

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

}

+ (void)setPublishableKey:(NSString *)key {
return [RadarSettings setPublishableKey:key];
Copy link

Copilot AI Dec 22, 2025

Choose a reason for hiding this comment

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

This method has a void return type, so it should not have a return statement. The return statement should be removed to match the pattern used by other setter methods in this file like setUserId and setDescription.

Suggested change
return [RadarSettings setPublishableKey:key];
[RadarSettings setPublishableKey:key];

Copilot uses AI. Check for mistakes.
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.

1 participant