-
Notifications
You must be signed in to change notification settings - Fork 166
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
Profile bindings #55
Profile bindings #55
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also add a UUID/unique identifier for each Theseus instance (mainly for easy navigation on the frontend)
theseus/src/api/profile_create.rs
Outdated
pub async fn profile_create_empty() -> crate::Result<Profile> { | ||
Ok(profile_create( | ||
String::from(DEFAULT_NAME), // the name/path of the profile | ||
String::from("1.8.2"), // the game version of the profile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.8.2 is a weird version IMO. why not 1.19.2 (which works on all platforms now)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like you forgot to change it
theseus/src/launcher/args.rs
Outdated
@@ -304,24 +299,29 @@ fn parse_arguments<F>( | |||
where | |||
F: Fn(&str) -> crate::Result<String>, | |||
{ | |||
let temporary_replace_char = "\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this used for? since it's used in multiple places as well it should be a constant and moved to the top of the file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should still be a const
…profile-bindings
…profile-bindings
theseus/src/api/profile_create.rs
Outdated
pub async fn profile_create_empty() -> crate::Result<Profile> { | ||
Ok(profile_create( | ||
String::from(DEFAULT_NAME), // the name/path of the profile | ||
String::from("1.8.2"), // the game version of the profile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like you forgot to change it
theseus/src/launcher/args.rs
Outdated
@@ -304,24 +299,29 @@ fn parse_arguments<F>( | |||
where | |||
F: Fn(&str) -> crate::Result<String>, | |||
{ | |||
let temporary_replace_char = "\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should still be a const
PR with base functionality so we can continue doing bindings + start using them- the actual calling of functions through tauri part is fairly simple.
Currently also trying to get more complex process API functions to be callable (ie: kill process, which might require minor API changes) + make the tauri functions in main.rs be modular, but not necessary for basic PR to start doing bindings
Fixes MOD-311