[5.0.0] Notification Click Listener - API update

[nan-li]

Description

One Line Summary

Notification Click Listener - API update

Details

Motivation

Notification Click Listener - API update

Scope

• Rename API around setNotificationOpenedHandler -> addClickListener, removeClickListener
• Also support adding multiple listeners, and removal.
• Rename object OSNotificationOpenedResult -> OSNotificationClickEvent
• Rename object OSNotificationAction -> OSNotificationClickResult
• Some small properties changes on those interfaces.

Public API

OneSignal.Notifications namespace
    + (void)addClickListener:(NSObject<OSNotificationClickListener>*_Nonnull)listener
    + (void)removeClickListener:(NSObject<OSNotificationClickListener>*_Nonnull)listener

OSNotificationClickListener
    - (void)onClickNotification:(OSNotificationClickEvent *_Nonnull)event
    func onClick(event: OSNotificationClickEvent) // For Swift API refined

OSNotificationClickEvent
    @property(readonly, nonnull)OSNotification* notification
    @property(readonly, nonnull)OSNotificationClickResult *result
    - (NSString* _Nonnull)stringify
    - (NSDictionary *_Nonnull)jsonRepresentation

OSNotificationClickResult
    @property(readonly, nullable)NSString* actionId
    @property(readonly, nullable)NSString* url

Testing

Manual testing

Test on physical iPhone 13 with iOS 16.x

• tested adding multiple click listeners and their removal
• tested accessing properties of the objects
• tested that click events are saved if there are no click listeners and then once a click listener is added, it is fired with the saved click events (existing behavior pre-user model).

Affected code checklist

• Notifications
  • Display
  • Open
  • Push Processing
  • Confirm Deliveries
• Outcomes
• Sessions
• In-App Messaging
• REST API requests
• Public API changes

Checklist

Overview

• I have filled out all REQUIRED sections above
• PR does one thing
• Any Public API changes are explained in the PR details and conform to existing APIs

Testing

• I have included test coverage for these changes, or explained why they are not needed
• All automated tests pass, or I explained why that is not possible
• I have personally tested this on my device, or explained why that is not possible

Final pass

• Code is as readable as possible.
• I have reviewed this PR myself, ensuring it meets each checklist item

---

This change is