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

CHNL-12941: Add badge count instructions to README #247

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

belleklaviyo
Copy link
Contributor

@belleklaviyo belleklaviyo commented Dec 20, 2024

Description

To separate out from work originally lumped in with #244
CHNL-12941

To be merged after 4.1.0 release

@belleklaviyo belleklaviyo changed the title Add badge count instructions to README CHNL-12941: Add badge count instructions to README Jan 2, 2025
@belleklaviyo belleklaviyo marked this pull request as ready for review January 2, 2025 14:55
@belleklaviyo belleklaviyo requested a review from a team as a code owner January 2, 2025 14:55
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@kennyklaviyo kennyklaviyo left a comment

Choose a reason for hiding this comment

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

this looks good to me, will you remove the README changes in #244 to avoid conflicts?

README.md Outdated
@@ -519,11 +493,24 @@ project setup with the code from the `KlaviyoSwiftExtension`. Below are instruct

Once you have these three things, you can then use the push notifications tester and send a local push notification to make sure that everything was set up correctly.

#### Badge Count
> ℹ️ Incrementing badge count support is available in SDK version [4.1.0](https://github.com/klaviyo/klaviyo-swift-sdk/releases/tag/4.1.0) and higher

Choose a reason for hiding this comment

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

To ensure people don't think they can set badge counts in earlier versions, should we say "Setting or incrementing the badge count is available..."

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.

5 participants