-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
fix(ivy): support multiple exportAs #27996
Conversation
You can preview 7b89b3e at https://pr27996-7b89b3e.ngbuilds.io/. |
7b89b3e
to
d97ab1f
Compare
You can preview d97ab1f at https://pr27996-d97ab1f.ngbuilds.io/. |
d97ab1f
to
f828c7f
Compare
You can preview f828c7f at https://pr27996-f828c7f.ngbuilds.io/. |
f828c7f
to
90d6a83
Compare
You can preview 90d6a83 at https://pr27996-90d6a83.ngbuilds.io/. |
@@ -117,7 +117,8 @@ function baseDirectiveFields( | |||
definitionMap.set('outputs', conditionallyCreateMapObjectLiteral(meta.outputs)); | |||
|
|||
if (meta.exportAs !== null) { | |||
definitionMap.set('exportAs', o.literal(meta.exportAs)); | |||
const exportAsValues = meta.exportAs.split(',').map(part => o.literal(part.trim())); |
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.
meta.exportAs
should already be split here. We should do that processing for AOT in ngtsc/annotations/src/directive.ts and for JIT in render3/jit/directive.ts.
For the compiler, it's actually a little more complicated as exportAs
is one of the metadata elements written to .d.ts
files, so that needs to change as well. Additionally there's the template type-checking code which matches against this...
Let me make you a precursor PR for the compiler which adds this support that you can build on.
90d6a83
to
d021ced
Compare
You can preview d021ced at https://pr27996-d021ced.ngbuilds.io/. |
Allows for multiple, comma-separated `exportAs` names, similarly to `ViewEngine`. These changes fix FW-708.
d021ced
to
a9a2519
Compare
You can preview a9a2519 at https://pr27996-a9a2519.ngbuilds.io/. |
Unacceptable!! ;) |
Allows for multiple, comma-separated `exportAs` names, similarly to `ViewEngine`. These changes fix FW-708. PR Close angular#27996
Allows for multiple, comma-separated `exportAs` names, similarly to `ViewEngine`. These changes fix FW-708. PR Close angular#27996
Allows for multiple, comma-separated `exportAs` names, similarly to `ViewEngine`. These changes fix FW-708. PR Close angular#27996
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Allows for multiple, comma-separated
exportAs
names, similarly toViewEngine
.These changes fix FW-708.