Skip to content
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

demo code for ticket 18521 #55

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

demo code for ticket 18521 #55

wants to merge 3 commits into from

Conversation

quirkles
Copy link

@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.

None yet

1 participant