Skip to content
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

(#106) multiple function call key support #107

Merged
merged 1 commit into from
Aug 8, 2023
Merged

(#106) multiple function call key support #107

merged 1 commit into from
Aug 8, 2023

Conversation

akeyder
Copy link
Contributor

@akeyder akeyder commented Aug 4, 2023

No description provided.

@akeyder akeyder self-assigned this Aug 4, 2023
tolga-coplu
tolga-coplu previously approved these changes Aug 4, 2023
Copy link
Contributor

@tolga-coplu tolga-coplu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -261,15 +325,23 @@ func (e *EngineEth) asyncSendRawTransaction(txsBytes []byte) (*string, error) {

// syncSendRawTransaction submits a raw transaction to engine synchronously
func (e *EngineEth) syncSendRawTransaction(txsBytes []byte) (*string, error) {
// check transaction data and return error if any issues (like low gas price or gas limit)
err := validateRawTransaction(txsBytes, e.Config.EngineConfig)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can add a deprecated comment on syncSendRawTransaction and asyncSendRawTransaction methods. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

tolga-coplu
tolga-coplu previously approved these changes Aug 8, 2023
Copy link
Contributor

@tolga-coplu tolga-coplu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

README.md Outdated
@@ -96,6 +96,8 @@ for detailed explanation and flags of each command see help
| endpoint.engine.nearNodeURL | ENDPOINT_ENGINE_NEARNODEURL | user should specify :heavy_exclamation_mark: | ❌ | URL of a Near Node to communicate |
| endpoint.engine.signer | ENDPOINT_ENGINE_SIGNER | user should specify :heavy_exclamation_mark: | ❌ | to be able to communicate with Near Node, user must have a Near account |
| endpoint.engine.signerKey | ENDPOINT_ENGINE_SIGNERKEY | user should specify :heavy_exclamation_mark: | ❌ | path to JSON file containing the Near Credentials. To be able to communicate with Near Node, user must have a Near signer key associated to account |
| endpoint.engine.functionKeyPrefixPattern | ENDPOINT_ENGINE_FUNCTIONKEYPREFIXPATTERN | `fk*.` | ❌ | prefix patter to search for key pair files of Function Call Keys |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo in prefix patter

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@akeyder akeyder merged commit 8cf55e8 into main Aug 8, 2023
2 checks passed
@akeyder akeyder deleted the dev/altug/#106 branch August 8, 2023 12:08
@akeyder akeyder linked an issue Sep 7, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

multiple function call key support
2 participants