Closed Pitasi closed 1 week ago
[!WARNING]
Rate limit exceeded
@Pitasi has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 24 minutes and 3 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between b3d31c689418f10f883af4e2cefc6e324309fa48 and 277759f7744f8559c9fbb2b9c8c1422be671707b.
This update introduces multiple new event messages related to intents and actions, specifically EventCreateIntent
, EventUpdateIntent
, EventCreateAction
, EventApproveAction
, and EventActionStateChange
. Additionally, it modifies how statuses and results are managed within actions, emphasizing method calls and context passing over direct assignments. These changes ensure more robust event handling and state management within the system.
File | Change Summary |
---|---|
CHANGELOG.md |
Updated summary of changes including new events and adjustments for specific parties to update SignatureRequest. |
proto/warden/intent/events.proto |
Added new protobuf messages for intent and action events: EventCreateIntent , EventUpdateIntent , EventCreateAction , EventApproveAction , EventActionStateChange . |
warden/x/intent/keeper/actions.go |
Changed methods to use SetStatus , SetResult , and AddApprover with context. |
warden/x/intent/keeper/msg_server_approve_action.go |
Updated ApproveAction method to use SetStatus and AddApprover methods with context. |
warden/x/intent/keeper/msg_server_new_action.go |
Added emission of EventCreateAction using sdkCtx.EventManager().EmitTypedEvent . |
warden/x/intent/keeper/msg_server_new_intent.go |
Added emission of EventCreateIntent in NewIntent function. |
warden/x/intent/keeper/msg_server_revoke_action.go |
Updated RevokeAction to set status using SetStatus method with context. |
warden/x/intent/keeper/msg_server_update_intent.go |
Added emission of EventUpdateIntent during the UpdateIntent function. |
warden/x/intent/types/action.go |
Updated SetResult , added SetStatus , and modified AddApprover methods to handle context and emit events. |
sequenceDiagram
participant User
participant MsgServer
participant Keeper
participant SDKContext
participant EventManager
User ->> MsgServer: NewIntent(request)
MsgServer ->> SDKContext: UnwrapContext()
MsgServer ->> Keeper: CreateIntent(ctx, request)
Keeper -->> MsgServer: Response(id)
MsgServer ->> EventManager: EmitTypedEvent(EventCreateIntent)
MsgServer -->> User: Response(id)
User ->> MsgServer: ApproveAction(request)
MsgServer ->> SDKContext: UnwrapContext()
MsgServer ->> Keeper: GetAction(ctx, id)
Keeper -->> MsgServer: Action
MsgServer ->> Keeper: SetStatus(ctx, ActionStatus)
Keeper -->> MsgServer: Status Set
MsgServer ->> Keeper: AddApprover(ctx, approver, timestamp)
Keeper -->> MsgServer: Approver Added
MsgServer ->> EventManager: EmitTypedEvent(EventApproveAction)
MsgServer -->> User: Response(success)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
See CHANGELOG.md for the list of emitted events.
Summary by CodeRabbit
New Features
EventCreateIntent
,EventUpdateIntent
,EventCreateAction
,EventApproveAction
, andEventActionStateChange
for better tracking of actions and intents.SimulateIntent
query request for enhanced simulation capabilities.Improvements
SignatureRequest
update mechanism to restrict access to specific parties.Bug Fixes