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

Add a get_static_info rpc for TUI API #483

Closed
mkundu1 opened this issue Jun 9, 2022 · 1 comment · Fixed by #651
Closed

Add a get_static_info rpc for TUI API #483

mkundu1 opened this issue Jun 9, 2022 · 1 comment · Fixed by #651
Assignees
Labels
enhancement Improve any current implemented feature

Comments

@mkundu1
Copy link
Contributor

mkundu1 commented Jun 9, 2022

Currently the tuigen is failing randomly during CI run with the following error:

2022-06-08 11:16:24,640 root         ERROR    GRPC_ERROR: basic_string::erase: __pos (which is 18446744073709551615) > this->size() (which is 0)

One possible issue is the huge number of API calls to get child-menu names and help-strings at each level. We can replace those with a single get_static_info rpc call similar to the datamodel and settings API.

@mkundu1 mkundu1 added enhancement Improve any current implemented feature Fluent labels Jun 9, 2022
@mkundu1 mkundu1 linked a pull request Jul 21, 2022 that will close this issue
@seanpearsonuk
Copy link
Collaborator

ADO 682015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve any current implemented feature
Projects
Status: 2021-2024
Development

Successfully merging a pull request may close this issue.

2 participants