Skip to content

refactor: simplify API - individualHooksWithVirtuals now acts as stan… - #4

Merged
diegopedro94 merged 1 commit into
v6.25.2-branchfrom
v6.25.2-with-api-simplification
Nov 14, 2025
Merged

diegopedro94 merged 1 commit into
v6.25.2-branchfrom
v6.25.2-with-api-simplification

Conversation

@diegopedro94

Copy link
Copy Markdown

…dalone flag

  • individualHooks: true now filters virtual columns by default (breaking change)
  • individualHooksWithVirtuals: true enables hooks + includes virtuals (old behavior)
  • Simplifies API: users only specify one flag instead of both
  • Based on codebase analysis: 99%+ of cases don't need virtual columns in hooks

Pull Request Checklist

  • Have you added new tests to prevent regressions?
  • If a documentation update is necessary, have you opened a PR to the documentation repository?
  • Did you update the typescript typings accordingly (if applicable)?
  • Does the description below contain a link to an existing issue (Closes #[issue]) or a description of the issue you are solving?
  • Does the name of your PR follow our conventions?

Description Of Change

Todos

…dalone flag

- individualHooks: true now filters virtual columns by default (breaking change)
- individualHooksWithVirtuals: true enables hooks + includes virtuals (old behavior)
- Simplifies API: users only specify one flag instead of both
- Based on codebase analysis: 99%+ of cases don't need virtual columns in hooks
@diegopedro94
diegopedro94 merged commit 8e39b9a into v6.25.2-branch Nov 14, 2025
10 of 48 checks passed
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.

1 participant