bkper-js 2.31.1 → 2.32.0

package/lib/index.d.ts

@@ -1585,6 +1585,12 @@ export declare class Book extends ResourceProperty<bkper.Book> {
      * @returns The name of the owner of the Book
      */
     getOwnerName(): string | undefined;
+    /**
+     * Gets the logo URL of the Book owner's custom domain, if any.
+     *
+     * @returns The logo URL, or undefined if the owner has no custom domain logo
+     */
+    getLogoUrl(): string | undefined;
     /**
      * Gets the permission for the current user in this Book.
      *

package/lib/model/Book.js

@@ -180,6 +180,14 @@ export class Book extends ResourceProperty {
     getOwnerName() {
         return this.payload.ownerName;
     }
+    /**
+     * Gets the logo URL of the Book owner's custom domain, if any.
+     *
+     * @returns The logo URL, or undefined if the owner has no custom domain logo
+     */
+    getLogoUrl() {
+        return this.payload.logoUrl;
+    }
     /**
      * Gets the permission for the current user in this Book.
      *
@@ -989,9 +997,27 @@ export class Book extends ResourceProperty {
             }
         }
     }
-    /** @internal */
+    /**
+     * Ensures every group has a cached `.accounts` Map after accounts are loaded.
+     *
+     * `linkAccountsAndGroups()` populates `.accounts` on groups that have accounts
+     * linked to them. Groups with zero accounts keep `.accounts = undefined`. This
+     * method fills that gap by initializing an empty Map on those groups.
+     *
+     * Without this, `Group.getAccounts()` sees `undefined`, skips the cache check,
+     * and falls through to a network request per group — causing hundreds of
+     * redundant `/groups/{id}/accounts` calls on books with many groups.
+     *
+     * IMPORTANT: This method must NOT be guarded by `allGroupsLoaded` or any
+     * similar flag. It runs inside `mapAccounts()`, which is always called after
+     * `mapGroups()` sets `allGroupsLoaded = true`. A guard like
+     * `if (!this.allGroupsLoaded)` would make this method dead code, silently
+     * reintroducing the redundant network requests.
+     *
+     * @internal
+     */
     ensureGroupsAccountMapsLoaded() {
-        if (this.idGroupMap && !this.allGroupsLoaded) {
+        if (this.idGroupMap) {
             for (const group of this.idGroupMap.values()) {
                 if (group.accounts == null) {
                     group.accounts = new Map();

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "bkper-js",
-    "version": "2.31.1",
+    "version": "2.32.0",
     "description": "Javascript client for Bkper REST API",
     "main": "./lib/index.js",
     "module": "./lib/index.js",
@@ -39,7 +39,7 @@
         "postversion": "git push --tags && yarn publish --new-version $npm_package_version && git push && echo \"Successfully released version $npm_package_version!\""
     },
     "dependencies": {
-        "@bkper/bkper-api-types": "^5.38.0",
+        "@bkper/bkper-api-types": "^5.39.1",
         "big.js": "^6.0.3",
         "dayjs": "^1.10.3",
         "luxon": "^1.25.0",