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.

Sign up for GitHub

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

Jump to bottom

Feature/str8 UI module calls 87 #93

Merged
merged 4 commits into from
Jan 30, 2025
Merged

Feature/str8 UI module calls 87 #93

merged 4 commits into from
Jan 30, 2025

Conversation

Skytrias
Copy link
Collaborator

Fixes #87
Not sure if the bindgen was correct? Also may have to rebuild the docs but I have no clue on that

@martinfouilleul
Copy link
Collaborator

Currently the api.json file should be updated manually (ie just add the prototypes for the str8 versions of the ui functions, you can copy/paste the non-str8 and modify the name and parameters).
The "generator" script is just a helper used when adding a lot of new functions, but it only outputs a skeleton (without docstrings)

@martinfouilleul
Copy link
Collaborator

(The docs should then be automatically generated from the api.json file)

@martinfouilleul martinfouilleul merged commit 7a465b4 into orca-app:main Jan 30, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martinfouilleul martinfouilleul Awaiting requested review from martinfouilleul

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add str8 versions of UI module calls
2 participants
@Skytrias @martinfouilleul