Remove redundant code and clarify type comments

Author: EmilianoSanchez

.github/workflows/ci-cd.yml

@@ -77,7 +77,7 @@ jobs:
             -Dsonar.pullrequest.base=${{ github.event.pull_request.base.ref }}
 
       - name: Store assets
-        if: github.event_name == 'push' && (github.ref == 'refs/heads/master' || github.ref == 'refs/heads/default_options')
+        if: github.event_name == 'push' && (github.ref == 'refs/heads/master' || github.ref == 'refs/heads/development')
         uses: actions/upload-artifact@v4
         with:
           name: assets
@@ -88,7 +88,7 @@ jobs:
     name: Upload assets
     runs-on: ubuntu-latest
     needs: build
-    if: github.event_name == 'push' && github.ref == 'refs/heads/default_options'
+    if: github.event_name == 'push' && github.ref == 'refs/heads/development'
     strategy:
       matrix:
         environment:

src/SplitClient.tsx

@@ -17,13 +17,7 @@ export function SplitClient(props: ISplitClientProps) {
   const context = useSplitClient(props);
 
   return (
-    <SplitContext.Provider value={{
-      ...context,
-      updateOnSdkReady: props.updateOnSdkReady ?? context.updateOnSdkReady,
-      updateOnSdkReadyFromCache: props.updateOnSdkReadyFromCache ?? context.updateOnSdkReadyFromCache,
-      updateOnSdkTimedout: props.updateOnSdkTimedout ?? context.updateOnSdkTimedout,
-      updateOnSdkUpdate: props.updateOnSdkUpdate ?? context.updateOnSdkUpdate
-    }}>
+    <SplitContext.Provider value={context}>
       {
         typeof children === 'function' ?
           children(context) :

src/types.ts

@@ -34,7 +34,7 @@ export interface ISplitStatus {
   isDestroyed: boolean;
 
   /**
-   * `lastUpdate` indicates when the most recent status event was emitted. This timestamp is only updated for events that are being listened to,
+   * `lastUpdate` indicates the timestamp of the most recent status event. This timestamp is only updated for events that are being listened to,
    * configured via the `updateOnSdkReady` option for `SDK_READY` event, `updateOnSdkReadyFromCache` for `SDK_READY_FROM_CACHE` event,
    * `updateOnSdkTimedout` for `SDK_READY_TIMED_OUT` event, and `updateOnSdkUpdate` for `SDK_UPDATE` event.
    */
@@ -47,29 +47,25 @@ export interface ISplitStatus {
 export interface IUpdateProps {
 
   /**
-   * `updateOnSdkUpdate` indicates if the hook or component will update (i.e., re-render) in case of an `SDK_UPDATE` event.
-   * If `true`, hooks or components consuming the context (such as `useSplitClient` and `useSplitTreatments`) will re-render on `SDK_UPDATE`.
+   * `updateOnSdkUpdate` indicates if the hook or component will update (i.e., re-render) or not in case of an `SDK_UPDATE` event.
    * It's value is `true` by default.
    */
   updateOnSdkUpdate?: boolean;
 
   /**
-   * `updateOnSdkTimedout` indicates if the hook or component will update (i.e., re-render) in case of a `SDK_READY_TIMED_OUT` event.
-   * If `true`, hooks or components consuming the context (such as `useSplitClient` and `useSplitTreatments`) will re-render on `SDK_READY_TIMED_OUT`.
+   * `updateOnSdkTimedout` indicates if the hook or component will update (i.e., re-render) or not in case of a `SDK_READY_TIMED_OUT` event.
    * It's value is `true` by default.
    */
   updateOnSdkTimedout?: boolean;
 
   /**
-   * `updateOnSdkReady` indicates if the hook or component will update (i.e., re-render) in case of a `SDK_READY` event.
-   * If `true`, hooks or components consuming the context (such as `useSplitClient` and `useSplitTreatments`) will re-render on `SDK_READY`.
+   * `updateOnSdkReady` indicates if the hook or component will update (i.e., re-render) or not in case of a `SDK_READY` event.
    * It's value is `true` by default.
    */
   updateOnSdkReady?: boolean;
 
   /**
-   * `updateOnSdkReadyFromCache` indicates if the hook or component will update (i.e., re-render) in case of a `SDK_READY_FROM_CACHE` event.
-   * If `true`, hooks or components consuming the context (such as `useSplitClient` and `useSplitTreatments`) will re-render on `SDK_READY_FROM_CACHE`.
+   * `updateOnSdkReadyFromCache` indicates if the hook or component will update (i.e., re-render) or not in case of a `SDK_READY_FROM_CACHE` event.
    * This params is only relevant when using `'LOCALSTORAGE'` as storage type, since otherwise the event is never emitted.
    * It's value is `true` by default.
    */