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(core) Move action resolution into a standalone class. #55523
Conversation
46f364f
to
ee52ed3
Compare
@@ -0,0 +1,352 @@ | |||
import {EventContract} from './eventcontract'; |
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.
do we have tests for this file?
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.
Existing eventcontract_test spec would cover the same, since it's used by EventContract
4b4a19c
to
f8756ae
Compare
dd2d566
to
23b7d79
Compare
TESTED=TGP |
23b7d79
to
e2945a6
Compare
Caretaker note: Patch cl/629442272 into sync CL for updates to internal BUILD rules. |
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.
reviewed-for: fw-core, primitives
8bc2073
to
1b79ac0
Compare
1b79ac0
to
07e000b
Compare
This refactor will enable moving this functionality into the dispatcher.
07e000b
to
333c63c
Compare
This PR was merged into the repository by commit 5903c37. |
…lar#55523) This refactor will enable moving this functionality into the dispatcher. PR Close angular#55523
This refactor will enable moving this functionality into the dispatcher.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Code is located in
EventContract
Issue Number: N/A
What is the new behavior?
Code is located in
ActionResolver
Does this PR introduce a breaking change?