Skip to content

fix: add getAfterResponse to overwrite by helper#2755

Merged
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:issues/2489
Aug 17, 2022
Merged

fix: add getAfterResponse to overwrite by helper#2755
mindlesscloud merged 2 commits into
apache:mainfrom
merico-ai:issues/2489

Conversation

@likyh
Copy link
Copy Markdown
Contributor

@likyh likyh commented Aug 17, 2022

Summary

fix afterResponse callback will overwrite by helper

Does this close any open issues?

close #2489

@likyh likyh changed the title fix: allow multi before/afterResponse fix: add getAfterResponse to overwrite by helper Aug 17, 2022
Copy link
Copy Markdown
Contributor

@mindlesscloud mindlesscloud left a comment

Choose a reason for hiding this comment

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

LGTM

@mindlesscloud mindlesscloud merged commit cd9775b into apache:main Aug 17, 2022
@likyh likyh deleted the issues/2489 branch October 23, 2022 10:48
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.

[Refactor][apiClient] It needs document or refactor in AfterResponse

2 participants