This repository has been archived by the owner on Aug 21, 2024. It is now read-only.
Changes to modify user hooks to allow various scenarios #9143
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
🤖 Generated by Copilot at 3f35e44
This pull request adds a new feature and some helper hooks to the user service. The feature allows skipping the hooks for internal calls by setting a
skipServiceHooks
param. The helper hooks areexecute-service-hooks
,is-path
, andis-skip-service-hooks
, which are used to conditionally run or skip hooks based on the context.References
closes #insert number here
Explanation
🤖 Generated by Copilot at 3f35e44
skipServiceHooks
property set to true (link)iffElse
function fromfeathers-hooks-common
to conditionally execute the existing hooks in theuser
service based on the result of theisSkipServiceHooks
function (link, link, link)🤖 Generated by Copilot at 3f35e44
QA Steps
List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.
Checklist