-
Notifications
You must be signed in to change notification settings - Fork 14
feat: Add structs for Rule, Condition, and Action #7 #18
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
Conversation
|
@codex review deeply |
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.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
apps/desktop/src-tauri/Cargo.toml
Outdated
| tauri-plugin-opener = "2" | ||
| serde = { version = "1", features = ["derive"] } | ||
| serde_json = "1" | ||
| uuid = { version = "1", features = ["v4"] } |
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.
Enable serde support for uuid models
The newly added Rule/Condition/Action structs derive Serialize/Deserialize and Rule includes a Uuid, but the uuid dependency is declared with only the v4 feature. Without the serde feature enabled, Uuid does not implement Serialize/Deserialize, so any code that tries to compile these models (once they are referenced) will fail with missing trait implementations. Include the serde feature in the dependency declaration so the derives build successfully.
Useful? React with 👍 / 👎.
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.
@POSTI-25 can you look into this?
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.
hey. done. let me know if any more changes are required
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Created structs for Rule, Condition and Action
Rule: Connects one or more conditions to a specific action.
Condition: Represents a single check against a part of a URL (like its host or path).
Action: Defines the outcome of a rule match, specifying which browser and profile to use.