Skip to content

Commit

Permalink
fix: Minor push/pull fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
jlacivita committed Feb 29, 2024
1 parent 8a46355 commit 0e8cd00
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Firebolt OpenRPC **MUST** support a method tag named `push-pull` that denotes a

If a *any* method has a `push-pull` tag then an additional method named `onRequest<method.Name>` **MUST** be auto-generated by the OpenRPC tooling; This new method is called the "pull provider method" for the remainder of this section.

The pull provider method **MUST** have the `x-result` extension schema set to the schema of the last parameter of the push method.
The pull provider method **MUST** have the `x-response` extension schema set to the schema of the last parameter of the push method.

The pull provider method **MUST** have any remaining parameters (not the last one) from the push method, as these are context parameters used by both push and pull.

Expand All @@ -42,7 +42,7 @@ The pull request method **MUST** have the `result` schema set to the schema of t

The pull request method **MUST** have any remaining parameters (not the last one) from the push method, as these are context parameters used by both push and pull.

The pull request method **MUST** have the `x-uses` array contain the value of the `x-provides` value from the push method.
The pull request method **MUST** have the `x-manages` array contain the value of the `x-provides` value from the push method.

If a *any* method has a `push-pull` tag then an additional method named `on<method.Name>` **MUST** be auto-generated by the OpenRPC tooling; This new method is called the "push notification method" for the remainder of this section.

Expand Down

0 comments on commit 0e8cd00

Please sign in to comment.