-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: Include request context in app-hooks #6894
Conversation
Uffizzi Preview |
c0cce4d
to
a363584
Compare
a363584
to
1e48b62
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- please add the reason why we need to add request context in PR summary.
@@ -19,10 +26,13 @@ export class FormColumnsController { | |||
async columnUpdate( | |||
@Param('formViewColumnId') formViewColumnId: string, | |||
@Body() formViewColumnbody: FormColumnUpdateReqType, | |||
|
|||
@Request() req: any, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can use RequestType
instead of any
? (apply to all)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will update this part as well in a followup pr... and inside hook I will use a custom type since in some case only certain fields we are passing ( user, headers, etc... )
At the moment we are not using it in OSS but later it will be helpful to extract data for Audit and listeners. |
Change Summary
Provide summary of changes with issue number if any.
Change type
Test/ Verification
Provide summary of changes.
Additional information / screenshots (optional)
Anything for maintainers to be made aware of