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

Adding a collapsed function from the CB improvements #11347

Open
AdRiley opened this issue Oct 17, 2024 · 2 comments
Open

Adding a collapsed function from the CB improvements #11347

AdRiley opened this issue Oct 17, 2024 · 2 comments
Labels
--bug Type: bug -gui

Comments

@AdRiley
Copy link
Member

AdRiley commented Oct 17, 2024

If I add a collapsed function from the CB it needs to

  • Have named placeholders for its arguments
  • Not be prefixed with the project name
image image

But expected is missing parameter names

@AdRiley AdRiley added the -gui label Oct 17, 2024
@AdRiley AdRiley added this to the 2024-10 Release milestone Oct 17, 2024
@farmaazon
Copy link
Contributor

The missing parameter names should be taken from MethodCall. The question is if we receive MethodCall in unprefixed call?

@farmaazon farmaazon added the --bug Type: bug label Oct 17, 2024
@farmaazon
Copy link
Contributor

Refinement notes:

  • Whatever the actual implementation would tell you, we want to have collapsed function in CB (but no local variables).
  • The lack of placeholders require investigation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
--bug Type: bug -gui
Projects
Status: 📤 Backlog
Development

No branches or pull requests

2 participants