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

feature[iOS]: Add support to live-activity notifications #802

Merged
merged 9 commits into from
Aug 17, 2024

Conversation

GonzaloAvilez
Copy link
Contributor

@GonzaloAvilez GonzaloAvilez commented Aug 11, 2024

GOAL:

  1. Adds support payload to live-activity based on apns2 library
  • dismissal-date
  • stale-date
  • timestamp
  • even
  • content-state
  1. Additional NOTES: Recently it was updated APNS2 with similar changes. Waiting new release of that library sideshow/apns2. new relase library -> https://github.com/sideshow/apns2/releases/tag/v0.24.0

    PR that support live activity in APNS2 -> Live Activity support added (revised and expanded) sideshow/apns2#219
    (My PR version to support live activity in APNS2) -> Add iOS 16 support for Live activity iOS notifications sideshow/apns2#223

  2. Updates in Readme iOS 16 + live activity feature

  3. Update go.mod and go.sum to for APNS2 0.24.0 library

TESTS

Test code coverage
image

Manual testing with curl POST events using GonzaloAvilez/apns2#1 with current suggested changes.
image

Test code coverage after using newest version of APNS2 within go modules:
image

@GonzaloAvilez GonzaloAvilez changed the title [iOS]: Add support payload to live-activity notifications feature[iOS]: Add support to live-activity notifications Aug 11, 2024
@GonzaloAvilez
Copy link
Contributor Author

@appleboy I would like to know what do you think about this PR proposal, it make sense to you support live activity notifications ?

@appleboy appleboy marked this pull request as ready for review August 17, 2024 00:44
@appleboy appleboy merged commit 4f10394 into appleboy:master Aug 17, 2024
3 of 4 checks passed
@appleboy
Copy link
Owner

@GonzaloAvilez Thanks.

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