-
Notifications
You must be signed in to change notification settings - Fork 692
feat: frontend improvements and updates #928
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
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.
Copilot reviewed 21 out of 24 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (9)
app/admin/api.py:77
- This comment appears to contain commented-out code.
# if not input.owner:
# raise IntentKitAPIError(
# status_code=400, key="BadRequest", message="Owner is required"
app/admin/api.py:85
- This comment appears to contain commented-out code.
# if user_id:
# if input.owner != user_id:
# raise IntentKitAPIError(
# status_code=400,
# key="BadRequest",
# message="Owner does not match user ID",
app/admin/api.py:92
- This comment appears to contain commented-out code.
# if user:
# max_fee += user.nft_count * 10
# if input.fee_percentage and input.fee_percentage > max_fee:
# raise IntentKitAPIError(
# status_code=400, key="BadRequest", message="Fee percentage too high"
app/admin/api.py:127
- This comment appears to contain commented-out code.
# if not input.owner:
# raise IntentKitAPIError(
# status_code=400, key="BadRequest", message="Owner is required"
app/admin/api.py:135
- This comment appears to contain commented-out code.
# if user_id:
# if input.owner != user_id:
# raise IntentKitAPIError(
# status_code=400,
# key="BadRequest",
# message="Owner does not match user ID",
app/admin/api.py:142
- This comment appears to contain commented-out code.
# if user:
# max_fee += user.nft_count * 10
# if input.fee_percentage and input.fee_percentage > max_fee:
# raise IntentKitAPIError(
# status_code=400, key="BadRequest", message="Fee percentage too high"
app/admin/credit.py:122
- This comment appears to contain commented-out code.
# class AdjustmentRequest(BaseModel):
# """Request model for adjusting a user account."""
# upstream_tx_id: Annotated[
# str, Field(str, description="Upstream transaction ID, idempotence Check")
# ]
# user_id: Annotated[str, Field(description="ID of the user to adjust")]
# credit_type: Annotated[CreditType, Field(description="Type of credit to adjust")]
# amount: Annotated[
# Decimal, Field(description="Amount to adjust (positive or negative)")
# ]
# note: Annotated[str, Field(description="Required explanation for the adjustment")]
intentkit/models/chat.py:385
- This method requires 1 positional argument, whereas overridden ChatMessageCreate.lambda requires 0.
datetime: lambda v: v.isoformat(timespec="milliseconds"),
intentkit/models/chat.py:562
- This method requires 1 positional argument, whereas overridden ChatCreate.lambda requires 0.
json_encoders={datetime: lambda v: v.isoformat(timespec="milliseconds")},
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| # Get the latest agent from create_or_update | ||
| latest_agent, agent_data = await deploy_agent(agent_id, agent, subject) | ||
| latest_agent, agent_data = await deploy_agent(agent_id, agent, "admin") |
Copilot
AI
Dec 27, 2025
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.
Variable latest_agent is not used.
|
|
||
| # Get the latest agent from create_or_update | ||
| latest_agent, agent_data = await deploy_agent(agent_id, agent, subject) | ||
| latest_agent, agent_data = await deploy_agent(agent_id, agent, "admin") |
Copilot
AI
Dec 27, 2025
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.
Variable agent_data is not used.
| latest_agent, agent_data = await deploy_agent(agent_id, agent, "admin") | |
| await deploy_agent(agent_id, agent, "admin") |
This PR includes: