Skip to content

feat: enhance BusinessBase with customizable where clause and operation modes#175

Merged
durlabhjain merged 5 commits into
mainfrom
feature/businessbase-books
Jun 2, 2026
Merged

feat: enhance BusinessBase with customizable where clause and operation modes#175
durlabhjain merged 5 commits into
mainfrom
feature/businessbase-books

Conversation

@durlabhjain

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 2, 2026 18:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances BusinessBase to support more extensible query generation by introducing operation “modes” and adding hook points that allow consumers to customize WHERE clauses and list query behavior without fully rewriting core methods.

Changes:

  • Added OperationMode constants and plumbed operationMode through load, lookupList, and list.
  • Extended createWhere to support a customizeWhere hook and updated call sites to pass richer context/options.
  • Refactored list generation to support getListStatement returning metadata, plus new hooks (addAdditionalColumns, customizeList) and a unified listResult return shape.

Comment thread lib/business/business-base.mjs Outdated
Comment thread lib/business/business-base.mjs Outdated
Comment thread lib/business/business-base.mjs Outdated
Comment thread lib/business/business-base.mjs Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

Comment thread lib/business/business-base.mjs
Comment thread lib/business/business-base.mjs
Comment thread lib/business/business-base.mjs Outdated
durlabhjain and others added 2 commits June 2, 2026 15:45
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 5 comments.

Comment thread lib/business/business-base.mjs
Comment thread lib/business/business-base.mjs
Comment thread lib/business/business-base.mjs
Comment thread lib/business/business-base.mjs
Comment thread lib/business/business-base.mjs
@durlabhjain durlabhjain merged commit 7544385 into main Jun 2, 2026
1 check passed
@durlabhjain durlabhjain deleted the feature/businessbase-books branch June 2, 2026 20:00
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