Skip to content

demo code for ticket 18521#55

Draft
quirkles wants to merge 3 commits intonrwl:mainfrom
quirkles:main
Draft

demo code for ticket 18521#55
quirkles wants to merge 3 commits intonrwl:mainfrom
quirkles:main

Conversation

@quirkles
Copy link
Copy Markdown

@quirkles quirkles commented Aug 8, 2023

reproduced the behaviour from ticket: the ticket this nrwl/nx#18521 in this fork.

If you remove the optional() call then the front end recognizes that the call without an input is invalid:
Screenshot 2023-08-07 at 9 13 45 PM

However the app builds absolutely fine:
Screenshot 2023-08-07 at 9 17 01 PM

And the call fails on the client
Screenshot 2023-08-07 at 9 17 16 PM

If you then add the optional() call, the ide recognizes its fixed, but the front end does not rebuild.

I would expect that the front end recognized the change to the client and rebuilds.

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.

1 participant