feat!(plugin): Remove SubscriptionCallback type (#259)

Author: Steven0351

plugin/src/definitions.ts

@@ -9,7 +9,7 @@ export interface PortalsPlugin {
   ): Promise<void>;
   addListener<T = unknown>(
     eventName: string,
-    listenerFunc: SubscriptionCallback<T>,
+    listenerFunc: (result: PortalMessage<T>) => void,
   ): Promise<PluginListenerHandle> & PluginListenerHandle;
 }
 
@@ -31,11 +31,3 @@ export interface PortalMessage<TData = any> {
   topic: string;
   data?: TData;
 }
-
-/**
- * The type definition from the callback running Portals.subscribe()
- */
-export type SubscriptionCallback<T = unknown> = (result: {
-  topic: string;
-  data: T;
-}) => void;

plugin/src/index.ts

@@ -5,7 +5,6 @@ import type {
   InitialContext,
   PortalMessage,
   PortalsPlugin,
-  SubscriptionCallback,
 } from './definitions';
 
 const Portals = registerPlugin<PortalsPlugin>('Portals', {
@@ -20,17 +19,18 @@ const Portals = registerPlugin<PortalsPlugin>('Portals', {
 export function getInitialContext<T = unknown>():
   | InitialContext<T>
   | undefined {
-  if (Capacitor.getPlatform() === "android") {
+  if (Capacitor.getPlatform() === 'android') {
+    // eslint-disable-next-line
     //@ts-ignore
-    return JSON.parse(AndroidInitialContext.initialContext())
+    return JSON.parse(AndroidInitialContext.initialContext());
   } else {
     return (window as any).portalInitialContext;
   }
 }
 
 export function subscribe<T = unknown>(
   topic: string,
-  callback: SubscriptionCallback<T>,
+  callback: (result: PortalMessage<T>) => void,
 ): Promise<PluginListenerHandle> {
   return Portals.addListener(topic, callback);
 }

plugin/src/web.ts

@@ -4,5 +4,5 @@ import type { PortalMessage, PortalsPlugin } from './definitions';
 
 export class PortalsWeb extends WebPlugin implements PortalsPlugin {
   // eslint-disable-next-line
-  async publishNative(_message: PortalMessage): Promise<void> { }
+  async publishNative(_message: PortalMessage): Promise<void> {}
 }